-
Notifications
You must be signed in to change notification settings - Fork 741
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 some nightly lints 2024-01-12 #3738
Conversation
.map(|p| p.value.0.clone()) | ||
.unwrap_or_else(|| syn::parse_str("::pyo3").unwrap()) | ||
match attr { | ||
Some(attr) => attr.value.0.clone(), |
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.
Here nightly clippy didn't like the attr.as_ref().map(|attr| attr.value.0.clone())
and wanted attr.clone()
, but I don't agree with that, because it's cloning more data than necessary.
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.
The new lint is way overly aggressive: rust-lang/rust-clippy#12135
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.
Agreed, it seems a bit broken. Thanks for filing that upstream 👍
Will just set this to merge and hope it goes in. |
CodSpeed Performance ReportMerging #3738 will degrade performances by 18.05%Comparing Summary
Benchmarks breakdown
|
7b28e26
to
4504a7c
Compare
Some lints are firing on latest nightly versions; hopefully this fixes them!