[gnugo-devel] important bugfix

[Gunnar Farneback]

This patch fixes a pattern bug that can cause crashes and improves
some related assertions. (It did cause a crash on NNGS, but the export
of crashed games to the webpage seems still not to be working. The bug
is obvious enough not to require an example, though.)

/Gunnar

Index: engine/move_reasons.c
===================================================================
RCS file: /cvsroot/gnugo/gnugo/engine/move_reasons.c,v
retrieving revision 1.72
diff -u -r1.72 move_reasons.c
--- engine/move_reasons.c       25 Mar 2002 07:39:18 -0000      1.72
+++ engine/move_reasons.c       30 Mar 2002 12:59:47 -0000
@@ -1396,6 +1396,9 @@
 
   ASSERT_ON_BOARD1(from);
   ASSERT_ON_BOARD1(to);
+  ASSERT1(board[from] == EMPTY, from);
+  ASSERT1(board[to] == EMPTY, to);
+
   cc = replacement_map[to];
 
   /* First check for an incompatible redistribution rule. */
Index: patterns/patterns.db
===================================================================
RCS file: /cvsroot/gnugo/gnugo/patterns/patterns.db,v
retrieving revision 1.61
diff -u -r1.61 patterns.db
--- patterns/patterns.db        26 Mar 2002 21:32:59 -0000      1.61
+++ patterns/patterns.db        30 Mar 2002 12:59:52 -0000
@@ -10476,6 +10476,7 @@
 
 Pattern ED88
 # db New pattern (3.1.28)
+# gf Corrected the replace action. (3.1.30)
 
 ??...X?          take territory on the fourth line
 oo..*OX
@@ -10494,7 +10495,7 @@
 -------
 
 ; lib(a)>2
-> replace(b,a)
+> replace(b,*)
 
 
 Pattern ED89