The Wayback Machine - https://web.archive.org/web/20200831022141/https://github.com/ornicar/chessground/pull/145
Skip to content

/ chessground

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.

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

use maps for state.pieces and other internals #145

Merged
merged 6 commits into from Jul 6, 2020
Merged

use maps for state.pieces and other internals #145

merged 6 commits into from Jul 6, 2020

Conversation

@niklasf
Copy link
Collaborator

niklasf commented Jul 5, 2020 •
edited

following veloce/lichobile@7cc3692 ff.

breaking change because of dests, and because lila also reads state.pieces
niklasf added 5 commits Jul 5, 2020
@niklasf
use map for state.pieces, following lichobile
74230ef
@niklasf
use maps also for square classes
a4a6e9f
@niklasf
use maps also for anims and other internals
c190c18
@niklasf
fix map empty check
efa6e3a
@niklasf
allow non-null assertions again
Loading status checks…
569ddd3
@niklasf niklasf force-pushed the pieces-map branch from 51be953 to 569ddd3 Jul 5, 2020
@niklasf
use map also for dests
Loading status checks…
f4b5229
@niklasf niklasf mentioned this pull request Jul 5, 2020
Open
1 of 3 tasks complete
@ornicar ornicar merged commit 661a038 into master Jul 6, 2020
2 checks passed
2 checks passed
build
Details
build
Details
@ornicar
Copy link
Owner

ornicar commented Jul 6, 2020

7.8.0 published
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants
@niklasf @ornicar
You can’t perform that action at this time.