wasmparser: clarify validator error message by keithw · Pull Request #2349 · bytecodealliance/wasm-tools
Navigation Menu
{{ message }}
bytecodealliance / wasm-tools Public
- Notifications You must be signed in to change notification settings
- Fork 321
Merged
alexcrichton merged 1 commit intobytecodealliance:mainfrom
Oct 13, 2025Merged
wasmparser: clarify validator error message#2349
alexcrichton merged 1 commit intobytecodealliance:mainfrom
wasmparser: clarify validator error message#2349
alexcrichton merged 1 commit intobytecodealliance:mainfrom
Conversation
Copy link
Contributor
keithw
commented
Oct 13, 2025
keithw
commented
This PR changes one validator error message to clarify that it's flagging an invalid (rather than malformed) memop alignment.
keithw
requested a review
from a team
as a code owner
keithw
requested review from
alexcrichton
and removed request for
a team
alexcrichton approved these changes Oct 13, 2025
Copy link
Member
alexcrichton
left a comment
alexcrichton
left a comment
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! CI failures look due to a new Rust nightly so I'll work on sorting that out, should be unrelated to this PR.
Copy link
Member
alexcrichton
commented
Oct 13, 2025
alexcrichton commented
Oct 13, 2025Ok if you could rebase now that #2350 is merged I can add this to the merge queue.
keithw
force-pushed
the
clarify-error
branch
from
cad1751 to
e594613
Compare
Copy link
Contributor Author
keithw
commented
Oct 13, 2025
keithw commented
Oct 13, 2025Sounds good, just done.
alexcrichton
added this pull request to the merge queue
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants