-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Fix extra_unused_lifetimes
false positive
#9037
Conversation
r? @dswij (rust-highfive has picked a reviewer for you, use r? to override) |
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.
Thanks for the fix!
Ideally, we'd have tests for this. But the change is small enough and doesn't break the current tests. I guess we can merge this in and add the tests in a separate PR.
@bors r+ |
📌 Commit 87eded6 has been approved by |
☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test |
This commit adds test for a `extra_unused_lifetimes` false positive from derive (rust-lang#9014). The fix for the FP is introduced in rust-lang#9037.
This commit adds test for a `extra_unused_lifetimes` false positive from derive (rust-lang#9014). The fix for the FP is introduced in rust-lang#9037.
I've added the test to cover for this in #9043 |
Nice! |
`extra_unused_lifetimes` add FP test case emitting from derived attributes. Add test to cover for #9014 which is fixed in #9037. changelog: [`extra_unused_lifetimes`] Add FP test case emitting from derived attributes. --- Seeing the FP from the test: ```sh $ git revert -m 1 1d1ae10 $ TESTNAME=extra_unused_lifetime cargo uitest ```
Sorry to post here, but is there any chance of this fix to also end up in/gets backported to 1.63? |
This PR fixes #9014.
I confirmed the FP on the
crates.io
source as @JohnTitor mentioned, and confirmed that the FP is no longer present following this change.I did not include a test in this PR because I think constructing one would be complicated, and the fix is pretty simple. But please let me know if this is unacceptable.
changelog: fix
extra_unused_lifetimes
FP