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:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
