Skip to content

Comments

fix panic when event pending on CALLBACK_CODE_YIELD#12190

Merged
alexcrichton merged 1 commit intobytecodealliance:mainfrom
dicej:fix-panic-on-yield-when-cancelled
Dec 22, 2025
Merged

fix panic when event pending on CALLBACK_CODE_YIELD#12190
alexcrichton merged 1 commit intobytecodealliance:mainfrom
dicej:fix-panic-on-yield-when-cancelled

Conversation

@dicej
Copy link
Contributor

@dicej dicej commented Dec 19, 2025

This replaces an invalid assertion with a more precise check when handling CALLBACK_CODE_YIELD. It also adds a test cover the scenario when a callback returns CALLBACK_CODE_YIELD when there's already an EVENT_CANCELLED pending.

This replaces an invalid assertion with a more precise check when handling
`CALLBACK_CODE_YIELD`.  It also adds a test cover the scenario when a callback
returns `CALLBACK_CODE_YIELD` when there's already an `EVENT_CANCELLED` pending.
@dicej dicej requested a review from alexcrichton December 19, 2025 22:59
@dicej dicej requested a review from a team as a code owner December 19, 2025 22:59
@github-actions github-actions bot added the wasmtime:api Related to the API of the `wasmtime` crate itself label Dec 20, 2025
@alexcrichton alexcrichton added this pull request to the merge queue Dec 22, 2025
Merged via the queue into bytecodealliance:main with commit df618ea Dec 22, 2025
45 checks passed
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