Skip to content

Implement async task cancellation #1264

Merged
alexcrichton merged 1 commit into
bytecodealliance:mainfrom
alexcrichton:async-cancel
Apr 30, 2025
Merged

Implement async task cancellation #1264
alexcrichton merged 1 commit into
bytecodealliance:mainfrom
alexcrichton:async-cancel

Conversation

@alexcrichton
Copy link
Copy Markdown
Member

@alexcrichton alexcrichton commented Apr 8, 2025

This commit implements async task cancellation in Rust for subtasks
created through imported calls. This involved some extra
bindings-related code to drop both lists and owned handles (effectively
a full destructor) along with some refactorings of how waitables are
handled due to new states that can pop up.

@alexcrichton alexcrichton changed the title Sketch out async task cancellation Implement async task cancellation Apr 30, 2025
@alexcrichton alexcrichton requested a review from dicej April 30, 2025 17:53
@alexcrichton alexcrichton marked this pull request as ready for review April 30, 2025 17:53
@alexcrichton
Copy link
Copy Markdown
Member Author

alexcrichton commented Apr 30, 2025

Ok with tests I think this is now good to go. I've currently based this on #1287 due to relying on a few refactorings there to get the tests working

@alexcrichton alexcrichton added this pull request to the merge queue Apr 30, 2025
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to a conflict with the base branch Apr 30, 2025
This commit implements async task cancellation in Rust for subtasks
created through imported calls. This involved some extra
bindings-related code to drop both lists and owned handles (effectively
a full destructor) along with some refactorings of how waitables are
handled due to new states that can pop up.
@alexcrichton alexcrichton enabled auto-merge April 30, 2025 20:57
@alexcrichton alexcrichton added this pull request to the merge queue Apr 30, 2025
Merged via the queue into bytecodealliance:main with commit 16ad298 Apr 30, 2025
18 of 19 checks passed
@alexcrichton alexcrichton deleted the async-cancel branch April 30, 2025 21:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants