bruno/brchess
1
0
Fork 0
Code Issues 9 Projects Releases Wiki Activity

Ensure EP flag is always valid #17

New Issue
Closed
opened 2024-05-27 14:57:47 +02:00 by bruno · 2 comments
bruno commented 2024-05-27 14:57:47 +02:00
Owner
Copy Link

Currently, e.p. validity is checked in pos_gen_pseudo(), as the moves will not be generated if no pawn can capture.

Better to do it in move_do() to handle position repetition.

Currently, e.p. validity is checked in `pos_gen_pseudo()`, as the moves will not be generated if no pawn can capture. Better to do it in `move_do()` to handle position repetition.
bruno added the
bug
 label 2024-05-27 14:57:47 +02:00
bruno added the
wontfix
 label 2024-05-28 10:05:15 +02:00
bruno commented 2024-05-28 10:06:15 +02:00
Author
Owner
Copy Link

No change, looks too expensive.

No change, looks too expensive.
bruno closed this issue 2024-05-28 10:06:15 +02:00
bruno reopened this issue 2024-06-09 22:27:02 +02:00
bruno removed the
wontfix
 label 2024-06-09 22:32:23 +02:00
bruno commented 2024-06-09 22:32:37 +02:00
Author
Owner
Copy Link

Even if useless pos->en_passant does not hurt, pos_ok() would fail.
Fixed in ebf01bc (move_do / move_do_alt).

Even if useless `pos->en_passant` does not hurt, `pos_ok()` would fail. Fixed in ebf01bc (`move_do` / `move_do_alt`).
bruno closed this issue 2024-06-09 22:32:37 +02:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
bug
Something is not working
duplicate
This issue or pull request already exists
enhancement
New feature
help wanted
Need some help
invalid
Something is wrong
question
More information is needed
wontfix
This won't be fixed
No Label
bug
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
bruno
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: bruno/brchess#17
No description provided.
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?