Skip to content

Conversation

@mateusz834
Copy link
Contributor

@mateusz834 mateusz834 commented Nov 8, 2025

This change removes the ref_start_index from the possible enum values of Index and OptionalIndex. It is not really a special purpose index, but a constant that tells the offset of static Refs. Also lets move it where such constant belongs i.e. to the Ref.

…ionalIndex

This change removes the ref_start_index from the possible enum values of
Index and OptionalIndex. It is not really a index, but a constant that
tells the offset of static Refs, so lets move it where such constant
belongs i.e. to the Ref.
@mateusz834 mateusz834 marked this pull request as draft November 8, 2025 19:38
@mateusz834 mateusz834 marked this pull request as ready for review November 8, 2025 21:03
@alexrp alexrp requested a review from mlugg November 8, 2025 21:07
Copy link
Member

@mlugg mlugg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I had been meaning to do exactly this change at some point, but it slipped my mind. Looks perfect; thank you!

@mlugg mlugg merged commit bdbfc7d into ziglang:master Nov 9, 2025
9 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.

2 participants