From: Rajarshi G. <rg...@in...> - 2008-03-22 13:35:30
|
-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 On Mar 22, 2008, at 2:59 AM, Egon Willighagen wrote: > Hi all, > > 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 - ------------------------------------------------------------------- Rajarshi Guha <rg...@in...> GPG Fingerprint: D070 5427 CC5B 7938 929C DD13 66A1 922C 51E7 9E84 - ------------------------------------------------------------------- A man is known by the company he organizes. -- Ambrose Bierce -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.8 (Darwin) iEYEARECAAYFAkflCxkACgkQZqGSLFHnnoSSuACg3byZtMvMnslUgc8T8rgtXkTe vvgAniQXBArcctd3Mcvo+t7EIYhjeUDi =04Dg -----END PGP SIGNATURE----- |