[gnugo-devel] move_safety revision

[Gunnar Farneback]

This patch revises the heuristics in examine_move_safety() not to
trust a cut_move move reason to imply safety. This fixes test case
strategy2:78 but may have as a side effect that GNU Go becomes
insufficiently aggressive and fails to cut when it should. Please keep
your eyes open for indications of this. The patch is in the CVS.

/Gunnar

Index: engine/move_reasons.c
===================================================================
RCS file: /cvsroot/gnugo/gnugo/engine/move_reasons.c,v
retrieving revision 1.22
diff -u -r1.22 move_reasons.c
--- engine/move_reasons.c       2001/10/04 18:38:53     1.22
+++ engine/move_reasons.c       2001/10/06 20:53:45
@@ -1607,9 +1607,10 @@
        what = move_reasons[r].what;
        switch (type) {
        case CUT_MOVE:
-         if (doing_scoring)
-           break;
-         /* otherwise fall through */
+         /* We don't trust cut moves, unless some other move reason
+           * indicates they are safe.
+          */
+         break;
        case SEMEAI_MOVE:
        case ATTACK_EITHER_MOVE:
        case DEFEND_BOTH_MOVE:    /* Maybe need better check for this case. */