Throw an exception when the given atom is not found in the chemmodel (fixes #3530861)
added as pull request in my name
Is IllegalArgumentException the appropriate exception here? Why not CDKException
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...
I wouldn't want to extend the CDKException hierarchy too much.
Signed off: https://github.com/johnmay/cdk/tree/patch/3530868
Oops... accidentally pushed to cdk-1.4.x.
Well, blame me if it gives trouble...
Log in to post a comment.
Sign up for the SourceForge newsletter:
You seem to have CSS turned off.
Please don't fill out this field.