-
Notifications
You must be signed in to change notification settings - Fork 93
Open
Description
Hi,
I am about to embark on an attempt to implement a chess engine written in c#. Researching if there are existing reusable WinBoard protocol implementations I came across the SharpChess codebase here on GitHub.
As I peeked into the code I found an apparent copy – paste bug around line 308 of Brain.cs.
The code reads:
if (this.MyPlayer.CanClaimInsufficientMaterialDraw)
{
WinBoard.SendDrawByFiftyMoveRule();
return;
}
But should read:
if (this.MyPlayer.CanClaimInsufficientMaterialDraw)
{
WinBoard.SendDrawByInsufficientMaterial();
return;
}
I can’t be arsed to fork the code and create a pull request, so I'm posting like this instead.
This is no showstopper as the draw will be claimed regardless. But the reason fo the claim might look strange to a human reader.
Regards!
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels