I agree with the patches - applied and pushed

Regarding implicit/explicit - would it be useful to check for implicit H's? I think updating the Javadocs will be easier

On Mon, Jun 25, 2012 at 3:17 PM, Egon Willighagen <egon.willighagen@gmail.com> wrote:
 
Now, obviously this is not supposed to happen, and indicates my
suspicion was not entirely unwarranted...

I checked the history of the file with 'git blame', and the test goes
back to the good old MFAnalyser days...

But, still, as the JavaDoc writes "The hydrogens must be implicit." I
am not sure what to do other than the attached 'fix' for the unit
test. Perhaps this JavaDoc was supposed to say "... must be
*ex*plicit"? Dunno...

So, I leave it up to you to give your second opinion.

Egon

--
Dr E.L. Willighagen
Postdoctoral Researcher
Department of Bioinformatics - BiGCaT
Maastricht University (http://www.bigcat.unimaas.nl/)
Homepage: http://egonw.github.com/
LinkedIn: http://se.linkedin.com/in/egonw
Blog: http://chem-bla-ics.blogspot.com/
PubList: http://www.citeulike.org/user/egonw/tag/papers

------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and
threat landscape has changed and how IT managers can respond. Discussions
will include endpoint security, mobile security and the latest in malware
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
Cdk-devel mailing list
Cdk-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/cdk-devel




--
Rajarshi Guha | http://blog.rguha.net
NIH Center for Advancing Translational Science