From 2d4e2bc62a4ee3fa93004755d4b93fa466d1e93e Mon Sep 17 00:00:00 2001 From: Marco Costalba Date: Mon, 10 Nov 2008 19:04:15 +0100 Subject: [PATCH] Fix in ok_to_history(): castle move is not a capture It is erroneusly considered a capture because king moves on the same square of the rook. Use the correct function Position::move_is_capture() instead of the open coded (and buggy) one. Signed-off-by: Marco Costalba --- src/search.cpp | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/src/search.cpp b/src/search.cpp index 5b4913c3..e28b866e 100644 --- a/src/search.cpp +++ b/src/search.cpp @@ -2155,13 +2155,11 @@ namespace { // ok_to_history() returns true if a move m can be stored - // in history. Should be a non capturing move. + // in history. Should be a non capturing move nor a promotion. bool ok_to_history(const Position& pos, Move m) { - return pos.square_is_empty(move_to(m)) - && !move_promotion(m) - && !move_is_ep(m); + return !pos.move_is_capture(m) && !move_promotion(m); } -- 2.39.2