-
Notifications
You must be signed in to change notification settings - Fork 574
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
Socket metadata update, please read my comments in description #2515
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: victoryang00 <yangyiwei2000@gmail.com>
Could you just commit the changed files? No need to commit so many files. |
@@ -9,6 +9,7 @@ | |||
*.a | |||
*.so | |||
.DS_Store | |||
foo |
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.
it is a local and private directory. right?
Fd restore memory yyds
My change is only at file
lib/wasm-micro-runtime/core/iwasm/interpreter/wasm_runtime.h
lib/wasm-micro-runtime/core/iwasm/libraries/libc-wasi/libc_wasi_wrapper.c
Notice there 146 files changes in this pull request so that mean some of change haven't been merge to main before. Please review changes in this files to make sure it is necessary. Thanks!