fix clippy::type_repetition_in_bounds

https://rust-lang.github.io/rust-clippy/master/index.html#/type_repetition_in_bounds

`CircularTupleWindows` even had a duplicate bound.

Co-Authored-By: Marcel Hellwig <[email protected]>

Author: hellow554

src/adaptors/coalesce.rs

@@ -17,9 +17,10 @@ where
     f: F,
 }
 
-impl<I: Clone, F: Clone, C> Clone for CoalesceBy<I, F, C>
+impl<I, F, C> Clone for CoalesceBy<I, F, C>
 where
-    I: Iterator,
+    I: Clone + Iterator,
+    F: Clone,
     C: CountItem<I::Item>,
     C::CItem: Clone,
 {

src/free.rs

@@ -166,10 +166,10 @@ where
 ///
 /// assert_eq!(cloned(b"abc").next(), Some(b'a'));
 /// ```
-pub fn cloned<'a, I, T: 'a>(iterable: I) -> iter::Cloned<I::IntoIter>
+pub fn cloned<'a, I, T>(iterable: I) -> iter::Cloned<I::IntoIter>
 where
     I: IntoIterator<Item = &'a T>,
-    T: Clone,
+    T: Clone + 'a,
 {
     iterable.into_iter().cloned()
 }

src/groupbylazy.rs

@@ -7,9 +7,9 @@ trait KeyFunction<A> {
     fn call_mut(&mut self, arg: A) -> Self::Key;
 }
 
-impl<A, K, F: ?Sized> KeyFunction<A> for F
+impl<A, K, F> KeyFunction<A> for F
 where
-    F: FnMut(A) -> K,
+    F: FnMut(A) -> K + ?Sized,
 {
     type Key = K;
     #[inline]
@@ -370,10 +370,12 @@ where
 ///
 /// See [`.group_by()`](crate::Itertools::group_by) for more information.
 #[must_use = "iterator adaptors are lazy and do nothing unless consumed"]
-pub struct Groups<'a, K: 'a, I: 'a, F: 'a>
+pub struct Groups<'a, K, I, F>
 where
-    I: Iterator,
+    I: Iterator + 'a,
     I::Item: 'a,
+    K: 'a,
+    F: 'a,
 {
     parent: &'a GroupBy<K, I, F>,
 }
@@ -409,10 +411,12 @@ where
 /// An iterator for the elements in a single group.
 ///
 /// Iterator element type is `I::Item`.
-pub struct Group<'a, K: 'a, I: 'a, F: 'a>
+pub struct Group<'a, K, I, F>
 where
-    I: Iterator,
+    I: Iterator + 'a,
     I::Item: 'a,
+    K: 'a,
+    F: 'a,
 {
     parent: &'a GroupBy<K, I, F>,
     index: usize,
@@ -537,9 +541,9 @@ where
 /// See [`.chunks()`](crate::Itertools::chunks) for more information.
 #[must_use = "iterator adaptors are lazy and do nothing unless consumed"]
 #[derive(Clone)]
-pub struct Chunks<'a, I: 'a>
+pub struct Chunks<'a, I>
 where
-    I: Iterator,
+    I: Iterator + 'a,
     I::Item: 'a,
 {
     parent: &'a IntoChunks<I>,
@@ -568,9 +572,9 @@ where
 /// An iterator for the elements in a single chunk.
 ///
 /// Iterator element type is `I::Item`.
-pub struct Chunk<'a, I: 'a>
+pub struct Chunk<'a, I>
 where
-    I: Iterator,
+    I: Iterator + 'a,
     I::Item: 'a,
 {
     parent: &'a IntoChunks<I>,

src/lib.rs

@@ -2124,8 +2124,7 @@ pub trait Itertools: Iterator {
     /// ```
     fn dropping_back(mut self, n: usize) -> Self
     where
-        Self: Sized,
-        Self: DoubleEndedIterator,
+        Self: Sized + DoubleEndedIterator,
     {
         if n > 0 {
             (&mut self).rev().nth(n - 1);
@@ -3962,7 +3961,7 @@ pub trait Itertools: Iterator {
     }
 }
 
-impl<T: ?Sized> Itertools for T where T: Iterator {}
+impl<T> Itertools for T where T: Iterator + ?Sized {}
 
 /// Return `true` if both iterables produce equal sequences
 /// (elements pairwise equal and sequences of the same length),

src/peeking_take_while.rs

@@ -96,17 +96,17 @@ where
 /// See [`.peeking_take_while()`](crate::Itertools::peeking_take_while)
 /// for more information.
 #[must_use = "iterator adaptors are lazy and do nothing unless consumed"]
-pub struct PeekingTakeWhile<'a, I: 'a, F>
+pub struct PeekingTakeWhile<'a, I, F>
 where
-    I: Iterator,
+    I: Iterator + 'a,
 {
     iter: &'a mut I,
     f: F,
 }
 
-impl<'a, I: 'a, F> std::fmt::Debug for PeekingTakeWhile<'a, I, F>
+impl<'a, I, F> std::fmt::Debug for PeekingTakeWhile<'a, I, F>
 where
-    I: Iterator + std::fmt::Debug,
+    I: Iterator + std::fmt::Debug + 'a,
 {
     debug_fmt_fields!(PeekingTakeWhile, iter);
 }

src/tuple_impl.rs

@@ -244,7 +244,7 @@ where
 /// information.
 #[must_use = "iterator adaptors are lazy and do nothing unless consumed"]
 #[derive(Debug, Clone)]
-pub struct CircularTupleWindows<I, T: Clone>
+pub struct CircularTupleWindows<I, T>
 where
     I: Iterator<Item = T::Item> + Clone,
     T: TupleCollect + Clone,

src/ziptuple.rs

@@ -39,8 +39,7 @@ pub struct Zip<T> {
 /// [`izip!()`]: crate::izip
 pub fn multizip<T, U>(t: U) -> Zip<T>
 where
-    Zip<T>: From<U>,
-    Zip<T>: Iterator,
+    Zip<T>: From<U> + Iterator,
 {
     Zip::from(t)
 }

tests/test_std.rs

@@ -364,9 +364,9 @@ fn test_rciter() {
 fn trait_pointers() {
     struct ByRef<'r, I: ?Sized>(&'r mut I);
 
-    impl<'r, X, I: ?Sized> Iterator for ByRef<'r, I>
+    impl<'r, X, I> Iterator for ByRef<'r, I>
     where
-        I: 'r + Iterator<Item = X>,
+        I: ?Sized + 'r + Iterator<Item = X>,
     {
         type Item = X;
         fn next(&mut self) -> Option<Self::Item> {