-
Notifications
You must be signed in to change notification settings - Fork 13.2k
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
ICE: clippy: can't get the span of an arbitrary parsed attribute
#137640
Comments
can't get the span of an arbitrary parsed attribute
cc @jdonszelmann proably :) |
@rustbot claim |
How in the world was there no test for this, this isn't even unusual usage lmao |
Because it's a clippy ICE presumably. |
there seems to be only a single test for that lint in clippy and perhaps it misses any of the "interesting" attributes.. if you run clippy on the rustc testsuite with this lint you will find this ICE several 100 times on all sorts of attrs though 🤷 |
Yeah, I'm just saying we definitely have a test for |
Now Clippy CI is running into this issue after the sync https://github.com/rust-lang/rust-clippy/actions/runs/13551226005/job/37875046333?pr=14300 😅 @jdonszelmann Let me know if you need help fixing this on the Clippy side. |
I've as of now spent no time fixing clippy. Feel free to, if you'd like otherwise I'll look at it later. Rn I'm trying to fix fundamental issues, while in this case it's not the api that's wrong but just clippy having a lint that uses part of the new and part of the old api and prolly the lint itself needs to change. Unsure why we didn't notice it earlier but it's what its |
…nishearth Clippy subtree update ? `@Manishearth` Cargo.lock change because of Clippy version bump and rustc_tool_utils new release. Fixes rust-lang#137640 Would be nice, if we could this merged before nightly is being build, so that this ICE is fixed tomorrow.
just throwing in that I have the same ICE on the same nightly version on a struct that uses a |
…nishearth Clippy subtree update ? `@Manishearth` Cargo.lock change because of Clippy version bump and rustc_tool_utils new release. Fixes rust-lang#137640 Would be nice, if we could this merged before nightly is being build, so that this ICE is fixed tomorrow.
…nishearth Clippy subtree update ? `@Manishearth` Cargo.lock change because of Clippy version bump and rustc_tool_utils new release. Fixes rust-lang#137640 Would be nice, if we could this merged before nightly is being build, so that this ICE is fixed tomorrow.
Code
Meta
rustc --version --verbose
:Error output
clippy-driver -Wclippy::macro-use-imports c.rs
Backtrace
The text was updated successfully, but these errors were encountered: