Assign improving only once

[stockfish] / src / search.cpp

diff --git a/src/search.cpp b/src/search.cpp
index 832501f2f70d94437625b35c836692b63f86c2fa..98e3e945d5537ac777d19ab4c608c63b74929c9e 100644 (file)
--- a/src/search.cpp
+++ b/src/search.cpp
@@ -654,6 +654,7 @@ namespace {
     if (inCheck)
     {
         ss->staticEval = eval = VALUE_NONE;
     if (inCheck)
     {
         ss->staticEval = eval = VALUE_NONE;

+        improving = true;

         goto moves_loop;
     }
     else if (ttHit)
         goto moves_loop;
     }
     else if (ttHit)


@@ -678,7 +679,6 @@ namespace {
     }
 
     improving =   ss->staticEval >= (ss-2)->staticEval
     }
 
     improving =   ss->staticEval >= (ss-2)->staticEval

-            /* || ss->staticEval == VALUE_NONE Already implicit in the previous condition */

                ||(ss-2)->staticEval == VALUE_NONE;
 
     if (skipEarlyPruning || !pos.non_pawn_material(pos.side_to_move()))
                ||(ss-2)->staticEval == VALUE_NONE;
 
     if (skipEarlyPruning || !pos.non_pawn_material(pos.side_to_move()))


@@ -816,9 +816,6 @@ moves_loop: // When in check, search starts from here
 
     MovePicker mp(pos, ttMove, depth, &thisThread->mainHistory, &thisThread->captureHistory, contHist, countermove, ss->killers);
     value = bestValue; // Workaround a bogus 'uninitialized' warning under gcc
 
     MovePicker mp(pos, ttMove, depth, &thisThread->mainHistory, &thisThread->captureHistory, contHist, countermove, ss->killers);
     value = bestValue; // Workaround a bogus 'uninitialized' warning under gcc

-    improving =   ss->staticEval >= (ss-2)->staticEval
-            /* || ss->staticEval == VALUE_NONE Already implicit in the previous condition */
-               ||(ss-2)->staticEval == VALUE_NONE;

 
     singularExtensionNode =   !rootNode
                            &&  depth >= 8 * ONE_PLY
 
     singularExtensionNode =   !rootNode
                            &&  depth >= 8 * ONE_PLY