Skip to content

Conversation

@xffxff
Copy link
Contributor

@xffxff xffxff commented Oct 2, 2021

Try to fix #8610
Related pr in ungrammer: rust-analyzer/ungrammar#41

T!['{'] => error_block(p, "expected ident"),
T![#] => {
let m = p.start();
attributes::outer_attrs(p);
Copy link
Contributor

Choose a reason for hiding this comment

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

Hm, seems like we can move this outer_attrs call to before the match? like we do in record_expr_field_list

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, the code will be cleaner.

bors bot added a commit to rust-analyzer/ungrammar that referenced this pull request Oct 3, 2021
41: add Attr to RestPat r=XFFXFF a=XFFXFF

Try to fix rust-lang/rust-analyzer#8610  
Related pr in rust-analyzer: rust-lang/rust-analyzer#10420

Co-authored-by: zhoufan <1247714429@qq.com>
@jonas-schievink
Copy link
Contributor

bors r+

@bors
Copy link
Contributor

bors bot commented Oct 6, 2021

@bors bors bot merged commit 94fa49c into rust-lang:master Oct 6, 2021
@lnicola lnicola changed the title Parse outer attributes on StructPatternEtCetera fix: Parse outer attributes on StructPatternEtCetera Oct 8, 2021
lnicola pushed a commit to lnicola/rust that referenced this pull request Nov 11, 2025
41: add Attr to RestPat r=XFFXFF a=XFFXFF

Try to fix rust-lang/rust-analyzer#8610  
Related pr in rust-analyzer: rust-lang/rust-analyzer#10420

Co-authored-by: zhoufan <1247714429@qq.com>
lnicola pushed a commit to lnicola/rust-analyzer that referenced this pull request Nov 15, 2025
41: add Attr to RestPat r=XFFXFF a=XFFXFF

Try to fix rust-lang#8610  
Related pr in rust-analyzer: rust-lang#10420

Co-authored-by: zhoufan <1247714429@qq.com>
lnicola pushed a commit to lnicola/rust that referenced this pull request Nov 15, 2025
41: add Attr to RestPat r=XFFXFF a=XFFXFF

Try to fix rust-lang/rust-analyzer#8610  
Related pr in rust-analyzer: rust-lang/rust-analyzer#10420

Co-authored-by: zhoufan <1247714429@qq.com>
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.

Parse outer attributes on StructPatternEtCetera

3 participants