clippy implied_bounds_in_impls - suppress globally?
Current nightly produces a lot of these:
warning: this bound is already specified as the supertrait of `DoubleEndedIterator`
--> crates/tor-dirmgr/src/event.rs:526:32
|
526 | fn statuses(&self) -> impl Iterator<Item = &DirStatus> + DoubleEndedIterator {
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#implied_bounds_in_impls
note: the lint level is defined here
--> crates/tor-dirmgr/src/lib.rs:9:9
|
9 | #![warn(clippy::all)]
| ^^^^^^^^^^^
= note: `#[warn(clippy::implied_bounds_in_impls)]` implied by `#[warn(clippy::all)]`
help: try removing this bound
|
526 - fn statuses(&self) -> impl Iterator<Item = &DirStatus> + DoubleEndedIterator {
526 + fn statuses(&self) -> impl DoubleEndedIterator<Item = &DirStatus> {
I disagree with this lint, at least in this case. All the existing cases in our codebase are just like this, with Iterator
. I prefer the Iterator + ExtraUsefulIterator
phrasing.
In my clippy fixes MR, I'm going to add suppressions at each site. But, I think we should add this to our global list of lint suppressions.