Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[spec] Clarify binary element bitfield meaning. #1761

Merged
merged 1 commit into from
Jun 26, 2024

Conversation

zstewar1
Copy link
Contributor

In the binary format, when treating the number distinguishing different kinds of element sections as a bitfield, the first bit distinguishes active segements from passive/declarative segments.

The previous phrasing used "indicate" to mean that a 1 bit signifies "passive or declarative", however it is easy to misread this as passive and declarative referring to the two possible values of the bit instead.

This change modifies the description of Bit 0 to make it clearer that it is differentiating active from passive or declarative, not passive from declarative, by using the "distinguishes ... from ..." language used in describing bit 1.

In the binary format, when treating the number distinguishing different
kinds of element sections as a bitfield, the first bit distinguishes
active segements from passive/declarative segments.

The previous phrasing used "indicate" to mean that a 1 bit signifies
"passive or declarative", however it is easy to misread this as passive
and declarative referring to the two possible values of the bit instead.

This change modifies the description of Bit 0 to make it clearer that it
is differentiating active from passive or declarative, not passive from
declarative, by using the "distinguishes ... from ..." language used in
describing bit 1.
@rossberg rossberg merged commit 2e6b128 into WebAssembly:main Jun 26, 2024
4 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