Permalink
Browse files

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 <mcostalba@gmail.com>
  • Loading branch information...
1 parent d89a03c commit 2d4e2bc62a4ee3fa93004755d4b93fa466d1e93e @mcostalba committed Nov 10, 2008
Showing with 2 additions and 4 deletions.
  1. +2 −4 src/search.cpp
View
@@ -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);
}

0 comments on commit 2d4e2bc

Please sign in to comment.