[gnugo-devel] trevorb:280

[Daniel Bump]

As I mentioned in an earlier if you run 3.1.24 with
the owl threats turned on you get a crash in trevorb:280.

The fault seems to be some new code that was added
in the patch arend_1_23.5 in the function reevaluate_ko_threats:

  for (k = 0; k <= 3; k++) {
    ko_stone = ko_move + delta[k];
    if (ON_BOARD(ko_stone) && countlib(ko_stone))
      break;
  }
  ASSERT_ON_BOARD1(ko_stone);

At move 96 of regression/games/trevor/auto/b16.sgf, this
turns up the incorrect ko stone. I suppose what was intended
was countlib(ko_stone)==1.

Dan

Index: engine/value_moves.c
===================================================================
RCS file: /cvsroot/gnugo/gnugo/engine/value_moves.c,v
retrieving revision 1.12
diff -u -r1.12 value_moves.c
--- engine/value_moves.c        2 Feb 2002 19:05:12 -0000       1.12
+++ engine/value_moves.c        5 Feb 2002 18:27:04 -0000
@@ -2330,7 +2330,7 @@
   /* Find the ko stone. */
   for (k = 0; k <= 3; k++) {
     ko_stone = ko_move + delta[k];
-    if (ON_BOARD(ko_stone) && countlib(ko_stone))
+    if (ON_BOARD(ko_stone) && countlib(ko_stone) == 1)
       break;
   }
   ASSERT_ON_BOARD1(ko_stone);