Skip to content

Correct attribute grammar in reference manual #14043

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

Merged
merged 1 commit into from
May 9, 2014

Conversation

sanxiyn
Copy link
Member

@sanxiyn sanxiyn commented May 8, 2014

Attribute grammar in reference manual allowed #[foo, bar], which does not match parser behavior.

Also rename nonterminals to match parser code.

Fix #13825.

@adrientetar
Copy link
Contributor

This closes #13825.

bors added a commit that referenced this pull request May 9, 2014
Attribute grammar in reference manual allowed `#[foo, bar]`, which does not match parser behavior.

Also rename nonterminals to match parser code.

Fix #13825.
@bors bors closed this May 9, 2014
@bors bors merged commit a4b7b8d into rust-lang:master May 9, 2014
@sanxiyn sanxiyn deleted the attr-reference branch May 9, 2014 15:43
bors added a commit to rust-lang-ci/rust that referenced this pull request Feb 13, 2023
…hanged, r=Veykril

fix: consider relative offset to fake ident token in expansion for completion

Fixes rust-lang#13836

When we check if the offset of fake ident token is out of bounds in the "actual expansion" (i.e. expansion without the fake ident token), we should take relative offset to it into account to match [how we'd get token after expansion](https://github.com/rust-lang/rust-analyzer/blob/f1b257f4eb4fef74b42fd7135d1cf3884e8b51c9/crates/ide-completion/src/context/analysis.rs#L53-L54).
bors pushed a commit to rust-lang-ci/rust that referenced this pull request Jan 28, 2025
I ran across this message while writing code and thought it was quite
odd. I've simplified it and hopefully made it clearer for non-native
English speakers.

changelog: clarify message for non-obvious precedence
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.

syntax::parse::attr implementation disagrees with BNF in manual
4 participants