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
Move stack switching logic into a new stack switching folder. #3987
Conversation
This is a minor rearrangment. I renamed "continuations" stuff to "stack_switching" and made a new folder for stack switching logic. I also added logic in esbuild.config.mjs to calculate which functions exported from stack_switching.mjs and automatically copy them to Module and into the Emscripten namespace. This makes unit testing a little neater since there is no modification of the Module object.
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.
One comment otherwise LGTM.
Makefile
Outdated
rm -fr node_modules | ||
shopt -s globstar && rm -fr src/**/*.o |
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.
On my Ubuntu system,
zsh: command not found: shopt
so it's bash specific. Not sure if it's worth having bash-specific commands here.
Similarly, MacOS has zsh so it will not work there.
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.
I switched to using find -delete
which should hopefully work more broadly.
Thanks for the review @rth! |
This is a minor rearrangment. I renamed "continuations" stuff to "stack_switching" and made a new folder for stack switching logic. I also added logic in esbuild.config.mjs to calculate which functions exported from stack_switching.mjs and automatically copy them to Module and into the Emscripten namespace. This makes unit testing a little neater since there is no modification of the Module object.
This is a minor rearrangment. I renamed "continuations" stuff to "stack_switching" and made a new folder for stack switching logic. I also added logic in esbuild.config.mjs to calculate which functions exported from stack_switching.mjs and automatically copy them to Module and into the Emscripten namespace. This makes unit testing a little neater since there is no modification of the Module object.
…e#3987) This is a minor rearrangment. I renamed "continuations" stuff to "stack_switching" and made a new folder for stack switching logic. I also added logic in esbuild.config.mjs to calculate which functions exported from stack_switching.mjs and automatically copy them to Module and into the Emscripten namespace. This makes unit testing a little neater since there is no modification of the Module object.
…e#3987) This is a minor rearrangment. I renamed "continuations" stuff to "stack_switching" and made a new folder for stack switching logic. I also added logic in esbuild.config.mjs to calculate which functions exported from stack_switching.mjs and automatically copy them to Module and into the Emscripten namespace. This makes unit testing a little neater since there is no modification of the Module object.
This is a minor rearrangment. I renamed "continuations" stuff to "stack_switching" and made a new folder for stack switching logic. I also added logic in esbuild.config.mjs to calculate which functions exported from stack_switching.mjs and automatically copy them to Module and into the Emscripten namespace. This makes unit testing a little neater since there is no modification of the Module object.
…e#3987) This is a minor rearrangment. I renamed "continuations" stuff to "stack_switching" and made a new folder for stack switching logic. I also added logic in esbuild.config.mjs to calculate which functions exported from stack_switching.mjs and automatically copy them to Module and into the Emscripten namespace. This makes unit testing a little neater since there is no modification of the Module object.
This is a minor rearrangment.
I renamed "continuations" stuff to "stack_switching" and made a new folder for stack switching logic. I also added logic in esbuild.config.mjs to calculate which functions exported from stack_switching.mjs and automatically copy them to Module and into the Emscripten namespace. This makes unit testing a little neater since there is no modification of the Module object.