Skip to content

Simplify some trap handling in wasmtime runtime#9673

Merged
alexcrichton merged 1 commit intobytecodealliance:mainfrom
alexcrichton:simplify-traps
Nov 25, 2024
Merged

Simplify some trap handling in wasmtime runtime#9673
alexcrichton merged 1 commit intobytecodealliance:mainfrom
alexcrichton:simplify-traps

Conversation

@alexcrichton
Copy link
Copy Markdown
Member

The needs_backtrace field on TrapReason is an old artifact of having the wasmtime and wasmtime-runtime crates split and there's no longer any need for that. This commit removes the field and directly queries it from the error as necessary when capturing backtraces.

The `needs_backtrace` field on `TrapReason` is an old artifact of having
the `wasmtime` and `wasmtime-runtime` crates split and there's no longer
any need for that. This commit removes the field and directly queries it
from the error as necessary when capturing backtraces.
@alexcrichton alexcrichton requested a review from a team as a code owner November 25, 2024 16:05
@alexcrichton alexcrichton requested review from pchickey and removed request for a team November 25, 2024 16:05
@github-actions github-actions bot added the wasmtime:api Related to the API of the `wasmtime` crate itself label Nov 25, 2024
Copy link
Copy Markdown
Contributor

@pchickey pchickey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 🍝

@alexcrichton alexcrichton added this pull request to the merge queue Nov 25, 2024
Merged via the queue into bytecodealliance:main with commit 6c97d5e Nov 25, 2024
@alexcrichton alexcrichton deleted the simplify-traps branch November 25, 2024 19:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

wasmtime:api Related to the API of the `wasmtime` crate itself

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants