WebAssembly / exception-handling

Proposal to add exception handling to WebAssembly
https://webassembly.github.io/exception-handling/
Other
162 stars 35 forks source link

Fix the description of the stack value for `catch` and `catch_all` #300

Closed JSMonk closed 7 months ago

JSMonk commented 7 months ago

Maybe I missed something, but it seems like the description of the stack value has a typo, and the catch_all should be instead of the catch_ref. If I'm wrong, feel free to close the PR 🤷‍♂️

rossberg commented 7 months ago

No, the text is correct. Both catch and catch_ref push the exception arguments to the stack, the other two don't; catch_ref pushes both arguments and exnref, catch_all neither.

JSMonk commented 7 months ago

@rossberg , yeap, I've already re-read it and understand what I understood wrong. But thank you for the clarification 🙏