Re: [gnugo-devel] fast_defense() improvement

[Gunnar Farnebäck]

Arend wrote:
> > So this is what the attached patch does. I have also rewritten the
> 
> And here it actually is.
> [...]
> +  static unsigned liberty_mark = -1;
> +  static unsigned lm[BOARDMAX];
> [...]
> +    if (++liberty_mark == 0)
> +      memset(lm, 0, sizeof(lm));

Unfortunately this code behaves differently the first time around; all
liberties are considered already marked then.

- bugfix in fast_defense()

/Gunnar

Index: engine/reading.c
===================================================================
RCS file: /cvsroot/gnugo/gnugo/engine/reading.c,v
retrieving revision 1.149
diff -u -r1.149 reading.c
--- engine/reading.c    8 Jun 2004 03:34:56 -0000       1.149
+++ engine/reading.c    17 Jun 2004 00:43:00 -0000
@@ -1299,8 +1299,10 @@
     /* What is the total number of liberties of the friendly strings around
      * the lunch?
      */
-    if (++liberty_mark == 0)
+    if (++liberty_mark == 0) {
       memset(lm, 0, sizeof(lm));
+      liberty_mark++;
+    }
     /* Loop over all neighbors of the lunch. */
     adj2 = chainlinks(adjs[j], adjs2);
     for (k = 0; k < adj2; k++) {