Skip to content

Add specialized IntMap for structured serialization - #465

Merged
Frotty merged 1 commit into
masterfrom
codex/int-map-serialization
Aug 12, 2026
Merged

Add specialized IntMap for structured serialization#465
Frotty merged 1 commit into
masterfrom
codex/int-map-serialization

Conversation

@Frotty

@Frotty Frotty commented Aug 12, 2026

Copy link
Copy Markdown
Member

Summary

  • add IntMap with direct integer keys and compiler-specialized typed value storage
  • replace the structured reader's legacy HashMap and IterableMap pair with one typed field map
  • make migration replacement and rename ownership explicit and correct
  • cover primitive, class, tuple, replacement, removal, clear/reuse, and reader-mutation behavior

Design

Wurst currently rejects mixed legacy/specialized generic declarations, while JASS hashtables require integer child keys. IntMap is therefore the smallest backend-aligned abstraction: the key needs no adapter and values avoid legacy integer erasure and typecasting.

Verification

  • grill typecheck --quiet
  • grill test --quiet
  • git diff --check

@Frotty

Frotty commented Aug 12, 2026

Copy link
Copy Markdown
Member Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Can't wait for the next one!

Reviewed commit: 43fd428de5

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@Frotty
Frotty merged commit 4891a00 into master Aug 12, 2026
2 checks passed
@Frotty
Frotty deleted the codex/int-map-serialization branch August 12, 2026 14:25
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