#1231 ChemModelManipulator.java the impossible happens

cdk-1.4.x
closed
5
2013-05-25
2012-05-30
Ralf Stephan
No

ChemModelManipulator.java:235:
// This should never happen.
return null;

Happens frequently when JCP calls with null atom (fixing there atm). Please throw something here.

Discussion

  • Confirmed. Patch being worked on.

    I suggest we start working on master, because this is an API change... in cdk-1.4.x we should not introduce API changes, but in master that is no problem.

    Ralf, is it possible to work on top of the master branch for the NG JChemPaint?

     
  • Ralf Stephan
    Ralf Stephan
    2012-05-30

    No problem, there is no work being done with master, save the pending pull request

     
  • Ralf Stephan
    Ralf Stephan
    2012-05-30

    ah you mean cdk master? my hands are off that

     
  • John May
    John May
    2013-05-23

    Resolved - see patch.

     
  • John May
    John May
    2013-05-25

    • status: open --> closed