#470 makes datadebug logger instances final static to class

cdk-1.4.x (181)

This patch is for a problem I found when working on another patch, and makes all logging tools final static. There is, I think, no reason why each instance should in fact have a separate logger instance. For this other patch, this one is crucial, as it allows constructors to call class methods, like setOrder(), causing the logging tool to be needed.

Otherwise, this patch does not fix a current bug.


  • Rajarshi Guha

    Rajarshi Guha - 2012-01-21

    Wouldn't a final static field make these classes non-threadable (unless calls to the logger synchronized?)

  • Egon Willighagen

    Yeah, good point. I updated the other patch, to no longer need this one, and thus withdrawing this patch.


Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:

No, thanks