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

Diagnostic warning despite #[allow(non_snake_case)] and incorrect fix for Greek characters #13521

Closed
vankessel opened this issue Oct 31, 2022 · 1 comment · Fixed by #14082
Closed
Labels
A-diagnostics diagnostics / error reporting C-bug Category: bug

Comments

@vankessel
Copy link

vankessel commented Oct 31, 2022

rust-analyzer version: 0.4.1265-standalone (ba28e19 2022-10-29)

rustc version: 1.64.0 (a55dd71d5 2022-09-19)

RA doesn't seem to take attributes into account. #[allow(non_snake_case)] is hidden behind the tooltip in this photo:

Screenshot 2022-10-30 191146

Also, the recommended fix also doesn't change anything. It should suggest lowercase delta, δ. This also happens for the other Greek characters like lamda. I suspect all non-ascii characters with lowercase and uppercase variants suffer from this.

Possibly related:
#8747
#12651

@lnicola lnicola added A-diagnostics diagnostics / error reporting C-bug Category: bug labels Oct 31, 2022
@lowr
Copy link
Contributor

lowr commented Feb 3, 2023

We currently don't have a means to handle attributes on expressions/statements. This is probably to be tracked in #6541.

As for the incorrect fix, that's a distinct problem and should be fixed separately.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-diagnostics diagnostics / error reporting C-bug Category: bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants