Re: [Cdk-devel] addAtom() contains contains()... necessary?

[Rajarshi G. <rx...@ps...>]

On Sat, 2006-06-17 at 16:05 +0200, Egon Willighagen wrote:
> On Saturday 17 June 2006 16:02, Rajarshi Guha wrote:
> > On Sat, 2006-06-17 at 15:46 +0200, Egon Willighagen wrote:
> > > Do we really need this check? We leave a lot of checking to the user in
> > > other places by policy too...
> >
> > It is a useful utility method. Why not keep it in there and note it as
> > an expensive method?
> 
> Mark the addAtom() as expensive??

Sorry I misread the original post.

What I wanted to say is that performing the check could be made optional
and if specified (via a boolean for example) it would result in addAtom
being an expensive (in terms of time) method.

-------------------------------------------------------------------
Rajarshi Guha <rx...@ps...> <http://jijo.cjb.net>
GPG Fingerprint: 0CCA 8EE2 2EEB 25E2 AB04 06F7 1BB9 E634 9B87 56EE
-------------------------------------------------------------------
All life evolves by the differential survival of replicating entities.
-- Dawkins