From: SourceForge.net <no...@so...> - 2009-03-06 15:19:46
|
Patches item #2668537, was opened at 2009-03-06 10:10 Message generated for change (Comment added) made by rajarshi You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=320024&aid=2668537&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: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: Egon Willighagen (egonw) Assigned to: Stefan Kuhn (shk3) Summary: Patch to split out SDFWriter from MDLWriter Initial Comment: Attached patch makes the split up, cleaning up the code. The new SDFWriter uses the MDLWriter for the mol blocks. Unit tests provided. The SDFWriter has the option to disable writing of properties, and always skips 2 internal CDK properties from output (see the unit tests). Patch intended for CDK 1.2, so two reviewers please. Patch is made with 'git format-patch', so should be easy to apply locally. ---------------------------------------------------------------------- >Comment By: Rajarshi Guha (rajarshi) Date: 2009-03-06 10:19 Message: Looks good to me ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=320024&aid=2668537&group_id=20024 |