#546 fix for the NPE in the DeduceBondSystemTool (bug #1260)

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

The attached patch solves the NullPointerException. It returns null, when the input was null. That does not seem to break anything in the code.

1 Attachments

Discussion

  • John May
    John May
    2012-10-14

    Looks good - I'm guessing the error was occurring when the list was empty so could have also checked for '.isEmpty()'.

    Signed commit is on my new accepted branch: cdk-1.4.x-accepted

    Can't change the milestone to accepted

     
  • John May
    John May
    2012-10-21

    • branch: --> master
    • milestone: Needs_Review --> Accepted
     
  • Applied and pushed into cdk-1.4.x.

     
    • status: open --> closed
    • branch: master --> cdk-1.4.x