Stream: git-wasmtime

Topic: wasmtime / Issue #1942 Upgrade wasmparser to 0.58.0


view this post on Zulip Wasmtime GitHub notifications bot (Jun 30 2020 at 00:02):

github-actions[bot] commented on Issue #1942:

Subscribe to Label Action

cc @bnjbvr, @fitzgen, @peterhuene

<details>
This issue or pull request has been labeled: "cranelift", "cranelift:wasm", "fuzzing", "lightbeam", "wasmtime:api"

Thus the following users have been cc'd because of the following labels:

To subscribe or unsubscribe from this label, edit the <code>.github/subscribe-to-label.json</code> configuration file.

Learn more.
</details>

view this post on Zulip Wasmtime GitHub notifications bot (Jun 30 2020 at 14:18):

alexcrichton commented on Issue #1942:

Thanks for this!

I think actually unimplemented! is appropriate in these places. These are all "safe unwraps" in that if the module linking proposal isn't enabled (which for wasmtime it never is) then these items will never come out of the parser and there's nothing to worry about.

It might be worth expanding the inline macro to something like unimplemented!("module linking not implemented yet") so it's more descriptive, but other than that this looks good to me.


Last updated: Dec 23 2024 at 13:07 UTC