Re: [Cdk-devel] [Cdk-jchempaint] LoggingTool

[Egon W. <ego...@gm...>]

Hi all,

I have uploaded a patch at:

http://sourceforge.net/tracker/?func=detail&aid=2816660&group_id=20024&atid=320024

On Fri, Jul 3, 2009 at 1:16 PM, Egon
Willighagen<ego...@gm...> wrote:
> I suggest this (major) refactoring in the CDK:
>
> 1. replace all use of Logger by an interface ILogger (with the same API)

ILoggingTool

> 2. implement ILogger Log4jLogger and SystemOutLogger

LoggingTool and SystemOutLoggingTool respectively.

> 3. set up a LoggerFactory to instantiate the Log4jLogger if the log4j
> library is found,
>   and default to SystemOutLogger if that lib is not found

Available in the LoggingToolFactory.

Additionally, it allows setting a custom ILoggingTool implementation
Class, to allow other logger frameworks.

Please give it a go, give comments, do a formal review, etc.

Looking forward to your experiences!

Egon


-- 
Post-doc @ Uppsala University
http://chem-bla-ics.blogspot.com/