Re: [Cdk-devel] patch 2902612

[Rajarshi G. <raj...@gm...>]

On Nov 29, 2009, at 4:05 PM, Egon Willighagen wrote:

> On Sun, Nov 29, 2009 at 8:51 PM, Rajarshi Guha <raj...@gm... 
> > wrote:
>
> The API and behavior of constructors is not really defined by an API.

Agreed - my question was more of whether there was a reason for  
defaults of 0

> Setting formal charge and other properties to zero likely makes sense
> in certain situations, and constructors is then the place to do this.


My understanding was that default values were always going to be  
UNSET, as opposed to 0. It looks like this one slipped by - unless  
there is a requirement for those 3 vars to be 0 by default.

Since it seems that there isn't a documented reason, I'll look into  
switching those to UNSET rather than 0

----------------------------------------------------
Rajarshi Guha        | NIH Chemical Genomics Center
http://www.rguha.net | http://ncgc.nih.gov
----------------------------------------------------
A method of solution is perfect if we can forsee from the start,
and even prove, that following that method we shall attain our aim.
	-- Leibnitz