From: Marco Costalba <mcostalba@gmail.com>
Date: Mon, 19 Jul 2010 11:28:33 +0000 (+0100)
Subject: Fix "pass ss->eval to qsearch()" condition
X-Git-Url: https://git.sesse.net/?p=stockfish;a=commitdiff_plain;h=0fb5d7a73792146d5e8422c993473f3deab18903;ds=sidebyside

Fix "pass ss->eval to qsearch()" condition

The seocond check is no more needed now and
anyhow is wrong to overwrite a TT entry if
present.

Spotted by Ralph Stoesser.

No functional change.

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

diff --git a/src/search.cpp b/src/search.cpp
index 20beb23a..b57d0400 100644
--- a/src/search.cpp
+++ b/src/search.cpp
@@ -1108,7 +1108,7 @@ namespace {
         && !pos.has_pawn_on_7th(pos.side_to_move()))
     {
         // Pass ss->eval to qsearch() and avoid an evaluate call
-        if (!tte || tte->static_value() == VALUE_NONE)
+        if (!tte)
             TT.store(posKey, ss->eval, VALUE_TYPE_EXACT, Depth(-127*OnePly), MOVE_NONE, ss->eval, ei.kingDanger[pos.side_to_move()]);
 
         Value rbeta = beta - razor_margin(depth);