From: Marco Costalba <mcostalba@gmail.com>
Date: Tue, 3 Jun 2014 10:38:58 +0000 (+0200)
Subject: Use see() instead of see_sign() in previous patch
X-Git-Url: https://git.sesse.net/?p=stockfish;a=commitdiff_plain;h=187a9fe5e7b8349b9eacf23e11cb801a32bb6b12;hp=6f48367094082ad03526f6ffe24d7c3873ff7386

Use see() instead of see_sign() in previous patch

Before it was working by accident in case of
see_sign() and failing with see() due to how
castle moves are coded (king captures the rook).

Better to explicitly filter out castling moves
and use see() without any surprise/trick.

No functional case.
---

diff --git a/src/search.cpp b/src/search.cpp
index cb502f07..9a6bd863 100644
--- a/src/search.cpp
+++ b/src/search.cpp
@@ -870,8 +870,9 @@ moves_loop: // When in check and at SpNode search starts from here
 
           // Decrease reduction for moves that escape a capture
           if (   ss->reduction
+              && type_of(move) == NORMAL
               && type_of(pos.piece_on(to_sq(move))) != PAWN
-              && pos.see_sign(make_move(to_sq(move), from_sq(move))) < 0)
+              && pos.see(make_move(to_sq(move), from_sq(move))) < 0)
               ss->reduction = std::max(DEPTH_ZERO, ss->reduction - ONE_PLY);
 
           Depth d = std::max(newDepth - ss->reduction, ONE_PLY);