[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [gnugo-devel] new_semeai()
|
From: |
Gunnar Farneback |
|
Subject: |
Re: [gnugo-devel] new_semeai() |
|
Date: |
Wed, 19 Feb 2003 21:52:49 +0100 |
|
User-agent: |
EMH/1.14.1 SEMI/1.14.3 (Ushinoya) FLIM/1.14.2 (Yagi-Nishiguchi) APEL/10.3 Emacs/20.7 (sparc-sun-solaris2.7) (with unibyte mode) |
Dan wrote:
> I've put up a patch called semeai_3_17.2 which is a
> complete rewrite of new_semeai(). I think it meets
> Gunnar's recent criticism of that function, making
> it symmetrical in its status updates, and unlike
> the old version, I think it is completely readable and
> maintainable.
Absolutely. This version looks a lot nicer than the old one.
> It does not do as well as the old version in the
> regressions, so I do not propose to add it to the
> CVS except after further revisions.
Do you have a regression delta? When I tried to patch up the old code
in a straight-forward way there were also more new failures than
passes.
/Gunnar