Skip to content

Fix insufficient material detection in Horde, add more test cases#51

Merged
veloce merged 5 commits intolichess-org:mainfrom
tom-anders:fixHorde
Feb 19, 2026
Merged

Fix insufficient material detection in Horde, add more test cases#51
veloce merged 5 commits intolichess-org:mainfrom
tom-anders:fixHorde

Conversation

@tom-anders
Copy link
Contributor

No description provided.

`setPieceAt` returns a new board but that was ignored here, leading to
the pawn not being removed, causing infinite recursion

add @useResult to make sure something like this does not happen again in
the future.
Found this by comparing with scalachess
This is closer to what Scalachess does and makes the code more readable.
_hordeBishops() now always returns the bishops of the Horde and
_piecesBishops always returns the bishops of the pieces.
@veloce veloce merged commit 919f40f into lichess-org:main Feb 19, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants