On Fri, Sep 06, 2002 at 09:50:05AM -0400, Berin Loritsch wrote: > > 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.
Yup. That makes sense. I will implement an alternate constructor. Regards, Michael Melhem > > > -- > To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]> > For additional commands, e-mail: <mailto:[EMAIL PROTECTED]> > -- To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]> For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
