From: SourceForge.net <no...@so...> - 2010-02-26 01:14:34
|
Patches item #2942196, was opened at 2010-01-29 07:29 Message generated for change (Settings changed) made by rajarshi You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=320024&aid=2942196&group_id=20024 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: None Group: Needs Review >Status: Closed >Resolution: Fixed Priority: 5 Private: No Submitted By: Stefan Kuhn (shk3) Assigned to: Rajarshi Guha (rajarshi) Summary: Uniform behavour of mdl(V2000)reader Initial Comment: The MDLReader l. 488 and the MDLV2000Reader l. 545 have different behaviour for the same situation. I think this is due to code duplication between these classes, they should share the reading of atoms and bonds. ---------------------------------------------------------------------- >Comment By: Rajarshi Guha (rajarshi) Date: 2010-02-25 20:14 Message: Looks good. Applied to cdk-1.2.x and pushed ---------------------------------------------------------------------- Comment By: Egon Willighagen (egonw) Date: 2010-02-24 15:24 Message: Rajarshi, can you please review the patch? ---------------------------------------------------------------------- Comment By: Egon Willighagen (egonw) Date: 2010-02-24 15:22 Message: Adding patch to synchronize behavior. ---------------------------------------------------------------------- Comment By: Stefan Kuhn (shk3) Date: 2010-01-29 10:47 Message: Please have a look at the code. The MDLV2000Reader removes 3d coordinates if all are 0,0,0, the MDLReader doesn't. It looks like the f (totalX == 0.0 && totalY == 0.0 && totalZ == 0.0) was added later, but forgotten in MDLReader. If nothing else is possible, we can make a static helper method handleCoordinates or so. ---------------------------------------------------------------------- Comment By: Egon Willighagen (egonw) Date: 2010-01-29 09:52 Message: Please describe the expected behavior which you like to see for both. (Code duplication is generally bad, indeed. Not so easy to fix here, as the format for the atom and bond block as actually different (line lengths), so sharing the same method is at least not possible...) ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=320024&aid=2942196&group_id=20024 |