Fuse iterators in InterleaveShortest

This brings InterleaveShortest in line with Interleave. The documentation already (incorrectly) stated that InterleaveShortest is fused.
rust-itertools · Feb 8, 2024 · d8f7f59 · d8f7f59
1 parent d5709c7
commit d8f7f59
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/src/adaptors/mod.rs b/src/adaptors/mod.rs
@@ -127,8 +127,8 @@ where
     I: Iterator,
     J: Iterator<Item = I::Item>,
 {
-    i: I,
-    j: J,
+    i: Fuse<I>,
+    j: Fuse<J>,
     next_coming_from_j: bool,
 }
 
@@ -139,8 +139,8 @@ where
     J: Iterator<Item = I::Item>,
 {
     InterleaveShortest {
-        i,
-        j,
+        i: i.fuse(),
+        j: j.fuse(),
         next_coming_from_j: false,
     }
 }
@@ -231,8 +231,8 @@ where
 
 impl<I, J> FusedIterator for InterleaveShortest<I, J>
 where
-    I: FusedIterator,
-    J: FusedIterator<Item = I::Item>,
+    I: Iterator,
+    J: Iterator<Item = I::Item>,
 {
 }
 

diff --git a/tests/quick.rs b/tests/quick.rs
@@ -1880,7 +1880,7 @@ quickcheck! {
 
     fn fused_interleave_shortest(a: Iter<i16>, b: Iter<i16>) -> bool
     {
-        !is_fused(a.clone().interleave_shortest(b.clone())) &&
+        is_fused(a.clone().interleave_shortest(b.clone())) &&
         is_fused(a.fuse().interleave_shortest(b.fuse()))
     }