#518 Throw an exception when the given atom is not found

Needs_Review
closed
nobody
cdk-1.4.x (181)
5
2012-10-28
2012-05-30
No

Throw an exception when the given atom is not found in the chemmodel (fixes #3530861)

Discussion

  • Rajarshi Guha

    Rajarshi Guha - 2012-07-01

    Is IllegalArgumentException the appropriate exception here? Why not CDKException

     
  • Egon Willighagen

    For master we can certainly do such things. Did we ever document properly the outcome of what we wanted to do with Exceptions in the new API? Only CDKException, or a proper hierarchy? I quite liked those RFCs... maybe we should try to restart that... possibly just in the wiki...

     
  • Rajarshi Guha

    Rajarshi Guha - 2012-07-02

    I wouldn't want to extend the CDKException hierarchy too much.

     
  • Egon Willighagen

    Oops... accidentally pushed to cdk-1.4.x.

    Well, blame me if it gives trouble...

     

Log in to post a comment.

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

Sign up for the SourceForge newsletter:

JavaScript is required for this form.





No, thanks