feat: add a public Range.iter() method (#18) (pubgrub-rs#187)

konstin · zanieb · web-flow · commit d2727138277f · 2024-03-11T21:07:33.000Z
Co-authored-by: Zanie Blue <contact@zanie.dev> Otherwise, it's not possible to implement custom formatting of `Range` types. This seems generally useful to expose. Example usages: https://github.com/astral-sh/uv/blob/8d721830db8ad75b8b7ef38edc0e346696c52e3d/crates/uv-resolver/src/pubgrub/report.rs#L97-L112 https://github.com/astral-sh/uv/blob/8d721830db8ad75b8b7ef38edc0e346696c52e3d/crates/uv-resolver/src/pubgrub/report.rs#L549-L560 https://github.com/astral-sh/uv/blob/8d721830db8ad75b8b7ef38edc0e346696c52e3d/crates/uv-resolver/src/pubgrub/report.rs#L568-L605 Upstream port of #18, but `impl Iterator<Item = (&Bound<V>, &Bound<V>)>` instead of `impl Iterator<Item = &(Bound<V>, Bound<V>)>` to avoid constraining it to a tuple. Co-authored-by: Zanie Blue <contact@zanie.dev>
diff --git a/src/range.rs b/src/range.rs
@@ -508,6 +508,11 @@ impl<V: Ord + Clone> Range<V> {
         }
         Self { segments }.check_invariants()
     }
+
+    /// Iterate over the parts of the range.
+    pub fn iter(&self) -> impl Iterator<Item = (&Bound<V>, &Bound<V>)> {
+        self.segments.iter().map(|(start, end)| (start, end))
+    }
 }
 
 impl<T: Debug + Display + Clone + Eq + Ord> VersionSet for Range<T> {

Original file line number	Diff line number	Diff line change
`@@ -508,6 +508,11 @@ impl<V: Ord + Clone> Range<V> {`
`508`	`508`	`}`
`509`	`509`	`Self { segments }.check_invariants()`
`510`	`510`	`}`
	`511`	`+`
	`512`	`+ /// Iterate over the parts of the range.`
	`513`	`+ pub fn iter(&self) -> impl Iterator<Item = (&Bound<V>, &Bound<V>)> {`
	`514`	`+ self.segments.iter().map(\|(start, end)\| (start, end))`
	`515`	`+ }`
`511`	`516`	`}`
`512`	`517`
`513`	`518`	`impl<T: Debug + Display + Clone + Eq + Ord> VersionSet for Range<T> {`