-
Notifications
You must be signed in to change notification settings - Fork 12.9k
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
Add identifier to unused import warnings #37394
Add identifier to unused import warnings #37394
Conversation
r? @eddyb (rust_highfive has picked a reviewer for you, use r? to override) |
r? @pnkfelix |
@@ -17,19 +17,19 @@ use std::mem::*; // shouldn't get errors for not using | |||
// everything imported | |||
|
|||
// Should get errors for both 'Some' and 'None' | |||
use std::option::Option::{Some, None}; //~ ERROR unused import | |||
//~^ ERROR unused import | |||
use std::option::Option::{Some, None}; //~ ERROR unused import `Some` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you forgot to add the colons after unused import
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Of all the silly things... Good catch!
Once the semi-colons have been added, it'll be good for me. |
54a73d7
to
2bd9418
Compare
@GuillaumeGomez Colons added 😄 |
Thanks! @bors: r+ rollup |
📌 Commit 2bd9418 has been approved by |
…h-id, r=GuillaumeGomez Add identifier to unused import warnings Fix rust-lang#37376. For some reason, though, I'm getting warnings with messages like "76:9: 76:16: unused import: `self::g`" instead of "unused import: `self::g`". @pnkfelix Any ideas what might be causing this?
Fix #37376.
For some reason, though, I'm getting warnings with messages like "76:9: 76:16: unused import:
self::g
" instead of "unused import:self::g
". @pnkfelix Any ideas what might be causing this?