-
Notifications
You must be signed in to change notification settings - Fork 13k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Parse default async fn
& Pre-expansion-gate default
#63749
Conversation
gate_all!(or_pattern_spans, or_patterns, "or-patterns syntax is experimental"); | ||
gate_all!(param_attrs, "attributes on function parameters are unstable"); | ||
gate_all!(let_chains, "`let` expressions in this position are experimental"); | ||
gate_all!(async_closure, "async closures are unstable"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
gate_all!(async_closure, "async closures are unstable"); | |
gate_all!(async_closures, "async closures are unstable"); |
Nit: everything else is plural or uncountable.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, you are right, but fixing it is not worth the churn of people switching to #![feature(async_closures)]
?
r=me with or without addressing the comment |
@bors r=petrochenkov |
📌 Commit b1a9e9e has been approved by |
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
Argh... smallvec is doing things it shouldn't... :( @bors r- |
Cherry picked out the 1st and the 3rd commits respectively for now. |
…etrochenkov Allow 'default async fn' to parse. - Parse default async fn. Fixes rust-lang#63716. (`cherry-pick`ed from 3rd commit in rust-lang#63749.) r? @petrochenkov
…etrochenkov Allow 'default async fn' to parse. - Parse default async fn. Fixes rust-lang#63716. (`cherry-pick`ed from 3rd commit in rust-lang#63749.) r? @petrochenkov
Not sure what is this blocked on, #63759 was merged, marking as waiting on author. |
@petrochenkov The pre-expansion gating for |
Ok. |
default
instead of post-expansion.default async fn
. Fixes [bug] async specialization is not parsed #63716.r? @petrochenkov