Skip to content

Merge wasip2/wasip3 reading/writing#756

Merged
alexcrichton merged 2 commits intoWebAssembly:mainfrom
alexcrichton:merge-wasip2-wasip3-read-write
Mar 3, 2026
Merged

Merge wasip2/wasip3 reading/writing#756
alexcrichton merged 2 commits intoWebAssembly:mainfrom
alexcrichton:merge-wasip2-wasip3-read-write

Conversation

@alexcrichton
Copy link
Collaborator

  • Generalize the wasip2_{read,write}_t types to wasi_{read,write}_t with conditional internals depending on the WASI version.
  • Fix error handling for stdin to use stdin error codes instead of filesystem error codes.
  • Call subtask.drop for writes to acknowledge the subtask being done.
  • When stdin is done, drop the future for the error after it's read to ensure the future isn't read twice.

@alexcrichton alexcrichton requested a review from dicej March 3, 2026 19:57
* Generalize the `wasip2_{read,write}_t` types to `wasi_{read,write}_t`
  with conditional internals depending on the WASI version.
* Fix error handling for stdin to use stdin error codes instead of
  filesystem error codes.
* Call `subtask.drop` for writes to acknowledge the subtask being done.
* When stdin is done, drop the future for the error after it's read to
  ensure the future isn't read twice.
@alexcrichton alexcrichton force-pushed the merge-wasip2-wasip3-read-write branch from c377a42 to 4f0cc5a Compare March 3, 2026 20:13
@alexcrichton alexcrichton force-pushed the merge-wasip2-wasip3-read-write branch from 7ce44ad to f967edb Compare March 3, 2026 20:42
@alexcrichton alexcrichton merged commit 3f7371f into WebAssembly:main Mar 3, 2026
56 of 58 checks passed
@alexcrichton alexcrichton deleted the merge-wasip2-wasip3-read-write branch March 3, 2026 23:04
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