[gnugo-devel] retire move_safety check

[Arend Bayer]

This does not seem necessary anymore, and it totatally makes sense to
have a territorial valuation for unsafe move (i.e., for sacrifice
moves).
Gives one PASS at trevord:620.

Arend


Index: engine/value_moves.c
===================================================================
RCS file: /cvsroot/gnugo/gnugo/engine/value_moves.c,v
retrieving revision 1.95
diff -u -p -r1.95 value_moves.c
--- engine/value_moves.c        29 May 2003 04:13:47 -0000      1.95
+++ engine/value_moves.c        2 Jun 2003 23:25:50 -0000
@@ -1933,8 +1933,11 @@ estimate_territorial_value(int pos, int
     strength[pos] = 0.0;
   }

-  /* tm added move_safety check (3.1.22) (see trevorc:880) */
-  if (does_block && move[pos].move_safety
+  /* We don't check for move safety here. This enables a territorial
+   * evaluation for sacrifice moves that enable a break-through (or
+   * an owl defense).
+   */
+  if (does_block
       && tryko(pos, color, "estimate_territorial_value", EMPTY, NO_MOVE)) {
     if (!retrieve_delta_territory_cache(pos, color, &this_value,
                                        &move[pos].influence_followup_value,