Re: [Cdk-devel] CDK properties and external properties

[Egon W. <ego...@gm...>]

On Sat, Mar 1, 2008 at 4:09 AM, Rajarshi Guha <rg...@in...> wrote:
> On Feb 29, 2008, at 10:00 PM, Egon Willighagen wrote:
>  >>  Does anybody object if I go ahead and add a prefix to the actual
>  >>  String values of CDK property names?
>  >
>  > An alternative would be to use a custom property object for
>  > SDFProperties, something like:
>  >
>  > public class SDFProperty extends String() {
>  > }
>  >
>  > Then you can detect SDF properties using instanceof...
>
>  As far as I can remember, we had decided that we didn't want to
>  create extra classes for properties.

Ah, did we... Mmmm, I wish we still had the RFC system operational...

>  Adding this new class means an update to the MDL*Reader codes - it
>  seems easier to just add a "cdk_" prefix to the String values of the
>  CDK property constants

OK, maybe not that bad. Simpler patch anyway... What about using the
XML Namespaces scheme, and use a "cdk:" prefix instead?

Egon

-- 
----
http://chem-bla-ics.blogspot.com/