-
Notifications
You must be signed in to change notification settings - Fork 590
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
Return error when shutdown() fails #2801
Return error when shutdown() fails #2801
Conversation
I've also added tests in WebAssembly/wasi-testsuite#94; will update the PR with the new link to WASI testsuite once the WASI testsuite PR is merged. |
6c0b7a2
to
458a747
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
@wenyongh could you have a look please? Thanks a lot |
LGTM. Should I merge it now? |
458a747
to
b1e52db
Compare
79b5c1d
to
1a4e123
Compare
1a4e123
to
d110c68
Compare
I've actually updated the PR to cover all platforms, not just POSIX; could you have another look, and if it's good, feel free to merge it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Fix issue reported in bytecodealliance#2787.
Related issue: #2787