alexcrichton opened PR #9325 from alexcrichton:enable-some-clippy-lints
to bytecodealliance:main
:
I was browsing the list of lints and enabled a few that seemed useful for cleaning up code after a refactoring.
alexcrichton requested fitzgen for a review on PR #9325.
alexcrichton requested wasmtime-compiler-reviewers for a review on PR #9325.
alexcrichton requested wasmtime-core-reviewers for a review on PR #9325.
alexcrichton requested wasmtime-default-reviewers for a review on PR #9325.
elliottt submitted PR review.
elliottt created PR review comment:
Was it intentional that the
.context
calls were all removed in this module?
elliottt submitted PR review:
I'm curious about the changes to the preview1 module removing all the
.context
calls, but other than that these all seem like great improvements :+1:
github-actions[bot] commented on PR #9325:
Subscribe to Label Action
cc @fitzgen, @saulecabrera
<details>
This issue or pull request has been labeled: "cranelift", "cranelift:area:aarch64", "wasi", "wasmtime:api", "wasmtime:ref-types", "winch"Thus the following users have been cc'd because of the following labels:
- fitzgen: wasmtime:ref-types
- saulecabrera: winch
To subscribe or unsubscribe from this label, edit the <code>.github/subscribe-to-label.json</code> configuration file.
Learn more.
</details>
alexcrichton submitted PR review.
alexcrichton created PR review comment:
Intentional yeah, turns out that the
try_into
in these cases was infallible and couldn't ever fail. That meant that the error case never happened and the.context(...)
calls were basically all dead code. I switched them all over to using?
which leverages theinto
conversion automatically which is known to never fail.
alexcrichton updated PR #9325.
alexcrichton has enabled auto merge for PR #9325.
alexcrichton merged PR #9325.
Last updated: Dec 23 2024 at 12:05 UTC