Learn how easy it is to sync an existing GitHub or Google Code repo to a SourceForge project! See Demo

Close

#418 Bug fix for a NullPointerException in the LoggingTool

Needs_Review
closed
nobody
cdk-1.4.x (181)
5
2012-10-28
2011-09-20
Egon Willighagen
No

The NullPointerException happens when a logger.debug(exception) is made with an exception with a 'null' message. This bug only manifests itself when -Dcdk.debugging=true is uses (i.e. debugging is turned on).

This patch splits out tests which should be run with and without debugging turned out into an "abstract" test class, with two extensions, each instantiating a LoggingTool in an appropriate way (with or without debugging on).

The second patch contains a unit test for the NPE, and the third patch a fix.

Discussion

  • Rajarshi Guha
    Rajarshi Guha
    2011-09-20

    Looks good. applied and pushed