Re: [gnugo-devel] More bad DRAGON2 accesses

[Trevor Morris]

Here's a patch to one of at least a few problems exposed
by this re-definition of DRAGON2:

http:/www.public32.com/games/go/trevor_1_27.1
 - re-order conditions in revise_semeai to insure valid DRAGON2 data

Index: engine/genmove.c
===================================================================
RCS file: /cvsroot/gnugo/gnugo/engine/genmove.c,v
retrieving revision 1.30
diff -u -r1.30 genmove.c
--- engine/genmove.c    22 Feb 2002 08:23:01 -0000      1.30
+++ engine/genmove.c    25 Feb 2002 05:57:16 -0000
@@ -20,6 +20,7 @@
  * Boston, MA 02111, USA.                                        *
 \* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */
 
+#include "gnugo.h"
 
 #include <stdio.h>
 #include <stdlib.h>
@@ -804,9 +805,9 @@
 
   for (pos = BOARDMIN; pos < BOARDMAX; pos++) {
     if (ON_BOARD(pos)
+       && dragon[pos].color == other
        && DRAGON2(pos).semeai
-       && dragon[pos].matcher_status == DEAD
-       && dragon[pos].color == other) {
+       && dragon[pos].matcher_status == DEAD) {
       found_one = 1;
       dragon[pos].matcher_status = UNKNOWN;
       if (dragon[pos].origin == pos)