Added some FIXME to track needed tests

This avoid us to forget some very needed tests now that
futility has changed in a whole big chunk we need to fine
tuning every splitted change.

No functional change.

Signed-off-by: Marco Costalba <mcostalba@gmail.com>

diff --git a/src/search.cpp b/src/search.cpp
index 2fece7cee10cf9e3891db8afe3641d845e09ed78..097243a0c73b45daa354f14985cc8e1ad12828f4 100644 (file)
--- a/src/search.cpp
+++ b/src/search.cpp
@@ -1411,9 +1411,14 @@ namespace {
         update_gains(pos, ss[ply - 1].currentMove, ss[ply - 1].eval, ss[ply].eval);
     }
 
-    // Post futility pruning
-    if (depth < SelectiveDepth && staticValue - PostFutilityValueMargin >= beta)
-        return (staticValue - PostFutilityValueMargin);
+    // Do a "stand pat". If we are above beta by a good margin then
+    // return immediately.
+    // FIXME: test with added condition 'allowNullmove || depth <= OnePly' and !value_is_mate(beta)
+    // FIXME: test with modified condition 'depth < RazorDepth'
+    if (  !isCheck
+        && depth < SelectiveDepth
+        && staticValue - PostFutilityValueMargin >= beta)
+        return staticValue - PostFutilityValueMargin;
 
     // Null move search
     if (    allowNullmove
@@ -1538,6 +1543,8 @@ namespace {
       movesSearched[moveCount++] = ss[ply].currentMove = move;
 
       // Futility pruning for captures
+      // FIXME: test disabling 'Futility pruning for captures'
+      // FIXME: test with 'newDepth < RazorDepth'
       Color them = opposite_color(pos.side_to_move());
 
       if (   !isCheck
@@ -1559,7 +1566,6 @@ namespace {
               continue;
       }
 
-
       // Futility pruning
       if (   !isCheck
           && !dangerous