Skip to content

refactor: make arena vacant list explicit - #268

Merged
tisonkun merged 1 commit into
apache:mainfrom
tisonkun:codex/clarify-arena-free-list
Sep 2, 2026
Merged

refactor: make arena vacant list explicit#268
tisonkun merged 1 commit into
apache:mainfrom
tisonkun:codex/clarify-arena-free-list

Conversation

@tisonkun

@tisonkun tisonkun commented Sep 2, 2026

Copy link
Copy Markdown
Member

Summary

  • represent the intrusive vacant-list head and links as Option<SlotId> instead of overloading slots.len() as a sentinel
  • rename next_vacant to vacant_head and document the slot ID lifetime directly
  • preserve the existing allocation, slot reuse, and stable-ID behavior without changing the size of the head or vacant-link fields because SlotId uses a NonZeroUsize niche

Validation

  • cargo x check
  • cargo x test
  • cargo x lint
  • interleaved baseline/branch benchmarks for completion and event cancellation remained flat

@tisonkun
tisonkun enabled auto-merge (squash) September 2, 2026 11:57
@tisonkun
tisonkun merged commit 2745639 into apache:main Sep 2, 2026
10 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