Skip to content

Forbid static async closures - #145607

Closed
GrigorenkoPV wants to merge 1 commit into
rust-lang:masterfrom
GrigorenkoPV:static-async-closure
Closed

Forbid static async closures#145607
GrigorenkoPV wants to merge 1 commit into
rust-lang:masterfrom
GrigorenkoPV:static-async-closure

Conversation

@GrigorenkoPV

@GrigorenkoPVGrigorenkoPV commented Aug 19, 2025

Copy link
Copy Markdown
Contributor

Discussed on Zulip.

This is technically a breaking change, but @compiler-errors says it's a bug that was never meant to be stable.

The only instance of static async in Rust code that I've found on GitHub that wasn't a javascript snippet in JS analyzers/generators is a rust-analyzer test.

@rustbot

Copy link
Copy Markdown
Collaborator

r? @lcnr

rustbot has assigned @lcnr.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

@rustbotrustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Aug 19, 2025
@compiler-errors

Copy link
Copy Markdown
Contributor

Sorry, already got that started in #145605

@GrigorenkoPV

Copy link
Copy Markdown
ContributorAuthor

Oh the misery

@rustbotrustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 19, 2025
@GrigorenkoPV
GrigorenkoPV deleted the static-async-closure branch August 19, 2025 14:35
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T-compilerRelevant to the compiler team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@GrigorenkoPV@rustbot@compiler-errors@lcnr