Skip to content

Conversation

Veykril
Copy link
Member

@Veykril Veykril commented Jul 20, 2024

Had to figure this out for rust-analyzer and noticed that the reference is lacking here (arguably not an important detail as 2015 is rather old :)

Sourced from rustc here: https://github.com/rust-lang/rust/blob/339015920d0787ea14aee05fca2c205683c525f9/compiler/rustc_parse/src/parser/ty.rs#L722-L731 and https://github.com/rust-lang/rust/blob/339015920d0787ea14aee05fca2c205683c525f9/compiler/rustc_parse/src/parser/ty.rs#L84-L99

bors added a commit to rust-lang/rust-analyzer that referenced this pull request Jul 20, 2024
Add missing dyn parse special cases in 2015 edition

There were a few more special cases to consider here -> rust-lang/reference#1538
Copy link
Contributor

@traviscross traviscross left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @Veykril. Looks good.

@traviscross traviscross added this pull request to the merge queue Jul 23, 2024
Merged via the queue into rust-lang:master with commit da188bd Jul 23, 2024
@Veykril Veykril deleted the ed-2015-dyn branch July 24, 2024 07:30
lnicola pushed a commit to lnicola/rust that referenced this pull request Jul 28, 2024
Add missing dyn parse special cases in 2015 edition

There were a few more special cases to consider here -> rust-lang/reference#1538
RalfJung pushed a commit to RalfJung/rust that referenced this pull request Aug 1, 2024
Add missing dyn parse special cases in 2015 edition

There were a few more special cases to consider here -> rust-lang/reference#1538
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