On Mon, 2005-06-20 at 18:22 +1200, Simon Kitching wrote:
> Hi,
>
> Currently the Log4JLogger class has this kind of thing all through it:
>
> if(is12) {
> getLogger().log(FQCN, (Priority) Level.DEBUG, message, null );
> } else {
> getLogger().log(FQCN, Level.DEBUG, message, null );
> }
>
> This code is intended to deal with the fact that log4j1.3 is both binary
> and source incompatible with 1.2 for code that calls the Logger.log
> method.
>
> I've had some discussion on the log4j list about this, but it appears
> that due to the way the migration from Category/Priority to Logger/Level
> was originally done they're a bit locked in to breaking compatibility so
> it looks like jcl really does need to handle the two differently.
>
> I would like to split the existing class into separate Log4J12Logger and
> Log4J13Logger parts, with both being part of the "autodiscovery"
> process. I think this will lead to more readable and maintainable code.
+1
however, may need to think about compatibility (class name specified on
the command line) and do something a little tricky.
- robert
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]