New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Patch/static analysis 2 #532
Conversation
now redundant variable.
I'll review it today or tomorrow. Sorry for the delay. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Generally fine and thanks for the important work for code cleanup, some of it taking advantage of Java improvements since the changed code was written. A few points, of only one item I just disagree with, which is about testing consistency of atom type lists. That assert was added to ensure we fully test a particular atom type list and I see no reason why to remove that.
Mostly busy work, my IDE lights up light a Christmas tree for so much of the CDK it's easy to ignore the warnings.