mirror of
https://github.com/sockspls/badfish
synced 2025-04-30 16:53:09 +00:00
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.
This commit is contained in:
parent
272936eaba
commit
2985a6b5d7
1 changed files with 3 additions and 6 deletions
|
@ -1189,16 +1189,13 @@ bool Position::has_game_cycle(int ply) const {
|
||||||
|
|
||||||
if (!(between_bb(s1, s2) & pieces()))
|
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)
|
if (ply > i)
|
||||||
return true;
|
return true;
|
||||||
|
|
||||||
// For nodes before or at the root, check that the move is a repetition one
|
// 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())
|
if (color_of(piece_on(empty(s1) ? s2 : s1)) != side_to_move())
|
||||||
continue;
|
continue;
|
||||||
|
|
||||||
|
|
Loading…
Add table
Reference in a new issue