> From: Michael Melhem [mailto:[EMAIL PROTECTED]] > > Hi, > > Is there a reason > org.apache.avalon.excalibur.monitor.Resource does not extend > from AbstractLogEnabled??
Yep. It's just one more thing the user has to remember. The Resource is a local object designed to represent a resource--with the proper methods to obtain it. Therefore, I suggest we allow for an alternate constructor: Resource( org.apache.avalon.logger.Logger ); If supplied, we will use it--otherwise we default to the NullLogger. > If not, then I will make that small change, because I think > that would be rather useful. I would prefer not to make that happen on an *Object*, as opposed to a component. If the user forgets to set the Logger, then we would get NullPointerExceptions where we didn't before. It is important we don't break things like that. -- To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]> For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
