clippy::iter_without_into_iter

CHANGELOG.md

@@ -14,6 +14,10 @@ Possible sections are:
 <!-- next-header -->
 ## [Unreleased] - ReleaseDate
 
+### Added
+
+- `IntoIterator` implementation for `FrozenMap` and `FrozenSet`
+
 ## [0.1.1] - 2024-01-15
 
 ### Changed

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "h3o-ice"
-version = "0.1.1"
+version = "0.1.2"
 authors = ["Sylvain Laperche <[email protected]>"]
 edition = "2021"
 description = "Frozen{Map,Set} for H3 cells, based on finite state transducers."

src/map.rs

@@ -393,6 +393,15 @@ impl FrozenMap<Vec<u8>> {
     }
 }
 
+impl<'a, D: AsRef<[u8]>> IntoIterator for &'a FrozenMap<D> {
+    type Item = (CellIndex, u64);
+    type IntoIter = FrozenMapIterator<'a>;
+
+    fn into_iter(self) -> Self::IntoIter {
+        self.iter()
+    }
+}
+
 // ------------------------------------------------------------------------------
 
 /// A builder for creating a frozen map.

src/set.rs

@@ -271,6 +271,15 @@ impl FrozenSet<Vec<u8>> {
     }
 }
 
+impl<'a, D: AsRef<[u8]>> IntoIterator for &'a FrozenSet<D> {
+    type Item = CellIndex;
+    type IntoIter = FrozenSetIterator<'a>;
+
+    fn into_iter(self) -> Self::IntoIter {
+        self.iter()
+    }
+}
+
 // ------------------------------------------------------------------------------
 
 /// A builder for creating a frozen set.

tests/integration/map.rs

@@ -89,7 +89,7 @@ fn load_from_bytes() {
 
     // Data is exactly the same.
     assert_eq!(
-        expected.iter().collect::<Vec<_>>(),
+        expected.into_iter().collect::<Vec<_>>(),
         result.iter().collect::<Vec<_>>()
     );
 }

tests/integration/set.rs

@@ -70,7 +70,7 @@ fn load_from_bytes() {
 
     // Data is exactly the same.
     assert_eq!(
-        expected.iter().collect::<Vec<_>>(),
+        expected.into_iter().collect::<Vec<_>>(),
         result.iter().collect::<Vec<_>>()
     );
 }