From: Egon W. <ego...@gm...> - 2008-03-22 13:47:26
|
On Sat, Mar 22, 2008 at 2:35 PM, Rajarshi Guha <rg...@in...> wrote: > > the 'fix' for toString() to only output non-NULL information lead me > > to the observation that new Bond() has the atom count set to 2... I do > > not believe that this is a good choice, but it is explicitly tested > > for in the unit tests... I still like to set the atom count for new > > Bond() to 0. > > I assume that you're referring to the default constructor (i.e., no > args). In that case I agree that the atom count should be 0 Yes, this refers to new Bond(). Egon -- ---- http://chem-bla-ics.blogspot.com/ |