Stream: git-wasmtime

Topic: wasmtime / PR #9321 bug: bindgen erraneously parses extra...

view this post on Zulip Wasmtime GitHub notifications bot (Sep 28 2024 at 15:08):

lwansbrough opened PR #9321 from lwansbrough:feature/paths to bytecodealliance:main:

Fixes an issue introduced in #9288 where an additional comma token is erroneously consumed. This meant if path was not the last property in the bindgen options, you would see an error "expected ,"

The provided tests were simple enough that path happened to be the last property. I haven't included a new test here because I think testing for such an issue is outside the scope of the tests.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 28 2024 at 15:08):

lwansbrough requested fitzgen for a review on PR #9321.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 28 2024 at 15:08):

lwansbrough requested wasmtime-core-reviewers for a review on PR #9321.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 28 2024 at 15:16):

lwansbrough closed without merge PR #9321.

Last updated: Dec 23 2024 at 13:07 UTC