From: Rajarshi G. <rg...@in...> - 2009-05-05 14:53:10
|
On May 5, 2009, at 10:43 AM, Egon Willighagen wrote: > On Tue, May 5, 2009 at 4:33 PM, Rajarshi Guha <rg...@in...> > wrote: >> Attached is a patch for the SMARTS parser bug 2786624, in which the >> Sc in >> Sc1ccccc1 is mistaken for scandium. > > The first string has 2 and 1 as ring closure, but no match. > > The test has not explicit testing that the SMARTS that is being parsed > has 7 atoms... and that it has S instead of Sc... Ah, of course. New patch puts the test in the proper JUnit sources (ParserTest) and makes the appropriate checks ------------------------------------------------------------------- Rajarshi Guha <rg...@in...> GPG Fingerprint: D070 5427 CC5B 7938 929C DD13 66A1 922C 51E7 9E84 ------------------------------------------------------------------- Q: What's polite and works for the phone company? A: A deferential operator. |