Skip to content
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 const_closures syntax. #13983

Merged
merged 1 commit into from
Jan 19, 2023

Conversation

onestacked
Copy link
Contributor

Enables parsing of the syntax for #![features(const_closures)] introduced in this PR

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jan 19, 2023
@Veykril
Copy link
Member

Veykril commented Jan 19, 2023

This line here also needs adjusting

Attr* ('for' GenericParamList)? 'static'? 'async'? 'move'? ParamList RetType?

(and then a cargo test -p syntax to update the syntax tree code, it should fail the first time you run it, second one should pass)

Enables parsing of the syntax for `#![features(const_closures)]` introduced in rust-lang/rust#106004
@onestacked
Copy link
Contributor Author

@Veykril Done

@Veykril
Copy link
Member

Veykril commented Jan 19, 2023

Thanks!
@bors r+

@bors
Copy link
Contributor

bors commented Jan 19, 2023

📌 Commit 8724085 has been approved by Veykril

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Jan 19, 2023

⌛ Testing commit 8724085 with merge bbb730a...

@bors
Copy link
Contributor

bors commented Jan 19, 2023

☀️ Test successful - checks-actions
Approved by: Veykril
Pushing bbb730a to master...

@bors bors merged commit bbb730a into rust-lang:master Jan 19, 2023
@onestacked onestacked deleted the parse_const_closure branch January 19, 2023 18:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants