Skip to content

MagicStack: use Multimap for Spells Cast#11154

Draft
Hanmac wants to merge 1 commit into
masterfrom
magicStackMultimap
Draft

MagicStack: use Multimap for Spells Cast#11154
Hanmac wants to merge 1 commit into
masterfrom
magicStackMultimap

Conversation

@Hanmac

@Hanmac Hanmac commented Jul 5, 2026

Copy link
Copy Markdown
Contributor

Still WIP

I'm refactoring how "ThisTurnCast" stuff is stored.
If it is done as ListMultimap, using Player as Map-Key, then we don't need extra counting.

and if we store ThisGameCast too, then we can clean up some other stuff.

@Hanmac Hanmac requested a review from tool4ever July 5, 2026 06:33
@Hanmac

Hanmac commented Jul 6, 2026

Copy link
Copy Markdown
Contributor Author

@tool4ever @MostCromulent
I want to find a way to copy the castThisTurn/LastTurn/ThisGame in GameCopier and GameSnapshot

for this i probably need to copy the LKI again to replace the oldPlayer with newPlayer (and maybe the host cards too?)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant