From: Joona Kiiski <joona.kiiski@gmail.com>
Date: Wed, 24 Feb 2010 18:13:35 +0000 (+0200)
Subject: Fix repetition detection bug
X-Git-Url: https://git.sesse.net/?p=stockfish;a=commitdiff_plain;h=e6f2d43b8a02aeb0180466d75be425800ec7c24d;hp=1a03f0b0d369637081e29a235afd0bce4241b6dd

Fix repetition detection bug

Bug spotted by Jouni Uski and fix suggested by Pablo Vazquez

Also add note that we are not always handling fifty move rule correctly

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

diff --git a/src/position.cpp b/src/position.cpp
index 2508db66..0336a832 100644
--- a/src/position.cpp
+++ b/src/position.cpp
@@ -1678,6 +1678,7 @@ Value Position::compute_non_pawn_material(Color c) const {
 /// Position::is_draw() tests whether the position is drawn by material,
 /// repetition, or the 50 moves rule. It does not detect stalemates, this
 /// must be done by the search.
+// FIXME: Currently we are not handling 50 move rule correctly when in check
 
 bool Position::is_draw() const {
 
@@ -1691,7 +1692,7 @@ bool Position::is_draw() const {
       return true;
 
   // Draw by repetition?
-  for (int i = 2; i < Min(Min(gamePly, st->rule50), st->pliesFromNull); i += 2)
+  for (int i = 4; i <= Min(Min(gamePly, st->rule50), st->pliesFromNull); i += 2)
       if (history[gamePly - i] == st->key)
           return true;