avalon-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Stephen McConnell" <mcconn...@osm.net>
Subject RE: Check this alternative to Loggable
Date Wed, 31 Oct 2001 15:12:24 GMT

Berin Loritsch wrote:
> Peter Royal wrote:
> > 
> > At 09:21 AM 10/31/2001 -0500, you wrote:
> > >It is in ${jakarta-avalon}/src/proposal/logger-v2/
> > 
> > ah, i was looking in ${jakarta-avalon}/proposal.
> > 
> > Why did you rename setLogger to enableLogging?  Just to prevent 
> > confusion?
> > 
> > otherwise looks good from the peanut gallery.
> > -pete
> 
> The main gist of the proposal is to migrate from using the
> LogKit Logger to the Logger interface included in Avalon
> Framework.  So yes, it is to prevent confusion.
> 
> Although, we can have both methods named setLogger().
> 
> That way, if folks do not have to rewrite anything--just
> change the import statement for Logger.

I would definately prefer to stay with "setLogger" for two reasons,
firstly, the change impact is less, and secondly, I don't see any 
confusion here.

Cheers, Steve.




--
To unsubscribe, e-mail:   <mailto:avalon-dev-unsubscribe@jakarta.apache.org>
For additional commands, e-mail: <mailto:avalon-dev-help@jakarta.apache.org>


Mime
View raw message