fix: VecIter using bounded RangeIter

starkware-libs · Jan 15, 2025 · 3bd1ff9 · 3bd1ff9
1 parent 12d7113
commit 3bd1ff9
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 5 deletions.
diff --git a/corelib/src/iter.cairo b/corelib/src/iter.cairo
@@ -230,4 +230,4 @@
 //! [`map`]: Iterator::map
 mod adapters;
 mod traits;
-pub use traits::{IntoIterator, Iterator};
+pub use traits::{FromIterator, IntoIterator, Iterator};
diff --git a/corelib/src/starknet/storage/vec.cairo b/corelib/src/starknet/storage/vec.cairo
@@ -422,7 +422,7 @@ pub impl VecIntoIterRange<
     }
     #[inline]
     fn into_iter_full_range(self: StoragePath<Vec<T>>) -> Self::IntoIter {
-        VecIter { current_index: (0..VecTraitImpl::len(self)).into_iter(), vec: self }
+        VecIter { current_index: (0..core::num::traits::Bounded::MAX).into_iter(), vec: self }
     }
 }
 
@@ -442,7 +442,7 @@ pub impl PathableVecIntoIterRange<
     #[inline]
     fn into_iter_full_range(self: T) -> Self::IntoIter {
         let vec = self.as_path();
-        VecIter { current_index: (0..vec.len()).into_iter(), vec }
+        VecIter { current_index: (0..core::num::traits::Bounded::MAX).into_iter(), vec }
     }
 }
 
@@ -473,7 +473,9 @@ pub impl MutableVecIntoIterRange<
     }
     #[inline]
     fn into_iter_full_range(self: StoragePath<Mutable<Vec<T>>>) -> Self::IntoIter {
-        MutableVecIter { current_index: (0..MutVecTraitImpl::len(self)).into_iter(), vec: self }
+        MutableVecIter {
+            current_index: (0..core::num::traits::Bounded::MAX).into_iter(), vec: self,
+        }
     }
 }
 
@@ -493,6 +495,6 @@ pub impl PathableMutableVecIntoIterRange<
     #[inline]
     fn into_iter_full_range(self: T) -> Self::IntoIter {
         let vec = self.as_path();
-        MutableVecIter { current_index: (0..vec.len()).into_iter(), vec }
+        MutableVecIter { current_index: (0..core::num::traits::Bounded::MAX).into_iter(), vec }
     }
 }