Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Factor out "physical" board state into a separate private struct #62

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Factor out "physical" board state into a separate private struct #62

wants to merge 6 commits into from

Commits on Jun 25, 2021

  1. Factor out "physical" board state into a separate private struct that… 

    … guarantees a valid board
    @terrorfisch
    terrorfisch committed Jun 25, 2021
    Configuration menu
    Copy the full SHA
    5efb818 View commit details
    Browse the repository at this point in the history

Commits on Jul 7, 2021

  1. Add more (safety) comments, rename methods and do not use unsafe in n… 

    …on-performance critical parts.
    @terrorfisch
    terrorfisch committed Jul 7, 2021
    Configuration menu
    Copy the full SHA
    ba1db23 View commit details
    Browse the repository at this point in the history

  2. Use MAX_PIECES_FOR_COLOR for move list length

    @terrorfisch
    terrorfisch committed Jul 7, 2021
    Configuration menu
    Copy the full SHA
    090fd06 View commit details
    Browse the repository at this point in the history

  3. Fix bugs in board_pieces

    @terrorfisch
    terrorfisch committed Jul 7, 2021
    Configuration menu
    Copy the full SHA
    46b80d5 View commit details
    Browse the repository at this point in the history

  4. Implement hash correctly and include it in board_pieces

    @terrorfisch
    terrorfisch committed Jul 7, 2021
    Configuration menu
    Copy the full SHA
    e301d95 View commit details
    Browse the repository at this point in the history

  5. Flag PieceType::legals as unsafe and document its soundness requirements

    @terrorfisch
    terrorfisch committed Jul 7, 2021
    Configuration menu
    Copy the full SHA
    a8d0ad2 View commit details
    Browse the repository at this point in the history