Re: [Cdk-devel] Patch #2207593

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

All,

On Thu, Dec 4, 2008 at 7:23 PM, Stefan Kuhn <ste...@eb...> wrote:
> could some people please have a look at that merge? It's mainly new tests and
> nothing spectacular.
> it's not too great if things are not taken up for months.

I have reviewed the patch, and Stefan updated the patch for my comments...

But because it contains a few (minor) API cleanups, for it the get
applied to cdk-1.2.x/ it needs another peer review by Miguel,
Christoph or Rajarshi.

I have 'grouped' the patch as (still) Needs Review which I describe in my blog:

http://chem-bla-ics.blogspot.com/2008/12/peer-reviewed-cheminformatics-2-code.html

Anyone can review, but the signoff has to be done by a senior developer...

Egon

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