[Cdk-devel] [ cdk-Patches-2668537 ] Patch to split out SDFWriter from MDLWriter

[SourceForge.net <no...@so...>]

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