Skip to content

make accepts_alpha_cache jumbled-only allocation#19

Merged
andy-k merged 1 commit intomainfrom
optional-jumbled-cache
Mar 28, 2026
Merged

make accepts_alpha_cache jumbled-only allocation#19
andy-k merged 1 commit intomainfrom
optional-jumbled-cache

Conversation

@andy-k
Copy link
Copy Markdown
Owner

@andy-k andy-k commented Mar 28, 2026

Summary

  • Change accepts_alpha_cache from always-allocated Box (8.3KB) to Option (None for classic, allocated on first jumbled use)
  • Classic-mode users pay zero heap cost for the jumbled cache
  • Add AlphaCacheEntry type alias to satisfy clippy complexity warning

Test plan

  • movegen baseline passes (classic)
  • jumbled mode autoplay runs correctly
  • cargo fmt, clippy, build pass

Change accepts_alpha_cache from always-allocated Box (8.3KB) to
Option<Box> (None for classic, allocated on first jumbled use).
Classic-mode users pay zero heap cost for the jumbled cache.

Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
@andy-k andy-k merged commit 1766ab0 into main Mar 28, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant