From 2985a6b5d7be8b813da528886ff11265bfac449b Mon Sep 17 00:00:00 2001
From: svivanov72 <48968697+svivanov72@users.noreply.github.com>
Date: Thu, 16 May 2019 15:14:11 +0300
Subject: [PATCH] Remove unused code (#2150)

Remove an unused operator in has_game_cycle (thanks @vondele)
and modify its comment to explain other code.

No functional change.
---
 src/position.cpp | 9 +++------
 1 file changed, 3 insertions(+), 6 deletions(-)

diff --git a/src/position.cpp b/src/position.cpp
index b8d7b106..edb40499 100644
--- a/src/position.cpp
+++ b/src/position.cpp
@@ -1189,16 +1189,13 @@ bool Position::has_game_cycle(int ply) const {
 
           if (!(between_bb(s1, s2) & pieces()))
           {
-              // In the cuckoo table, both moves Rc1c5 and Rc5c1 are stored in the same
-              // location. We select the legal one by reversing the move variable if necessary.
-              if (empty(s1))
-                  move = make_move(s2, s1);
-
               if (ply > i)
                   return true;
 
               // For nodes before or at the root, check that the move is a repetition one
-              // rather than a move to the current position
+              // rather than a move to the current position.
+              // In the cuckoo table, both moves Rc1c5 and Rc5c1 are stored in the same
+              // location, so we have to select which square to check.
               if (color_of(piece_on(empty(s1) ? s2 : s1)) != side_to_move())
                   continue;
 
-- 
2.39.2