From: SourceForge.net <no...@so...> - 2009-01-03 14:11:24
|
Patches item #2483731, was opened at 2009-01-03 15:11 Message generated for change (Tracker Item Submitted) made by Item Submitter You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=320024&aid=2483731&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: Open Resolution: None Priority: 5 Private: No Submitted By: Egon Willighagen (egonw) Assigned to: Nobody/Anonymous (nobody) Summary: factoring out of ioformats Initial Comment: The CDK IO format definitions and matchers are quite useful outside the scope of just the CDK library, for example, for the core of Bioclipse. The IChemFormatMatcher implementations indeed do not depend on 'interfaces' or 'core' modules. Only complication is that TestMethod and TestClass are being used, as well as DataFeatures. I have moved these into a second new module 'annotation' as they deal with meta data. The branch egonw-ioformats is derived from cdk1.2.x/ and contains the necessary updates: * three new modules: annotation, ioformats, test-ioformats * new .cdkdepends etc for these * @cdk.module updates * replaces a use of MathTools.isOdd() with a local copy (one liner) This patch is ready for review by two senior reviewers, as I like to see it included in the CDK 1.2.0 release. Attached is a PNG with the new module dependency graph. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=320024&aid=2483731&group_id=20024 |