#1231 ChemModelManipulator.java the impossible happens

cdk-1.4.x
closed
5
2013-05-25
2012-05-30
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

  • Egon Willighagen

    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
     

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

Sign up for the SourceForge newsletter:





No, thanks