Cut or ignore ranges that exceed max_lines

Author: cruessler

gix-blame/src/file/tests.rs

@@ -1080,6 +1080,30 @@ mod blame_ranges {
         assert_eq!(ranges.to_zero_based_exclusive_ranges(100), vec![0..10]);
     }
 
+    #[test]
+    fn to_zero_based_exclusive_ignores_range_past_max_lines() {
+        let mut ranges = BlameRanges::from_one_based_inclusive_range(1..=5).unwrap();
+        ranges.add_one_based_inclusive_range(16..=20).unwrap();
+
+        assert_eq!(ranges.to_zero_based_exclusive_ranges(7), vec![0..5]);
+    }
+
+    #[test]
+    fn to_zero_based_exclusive_range_doesnt_exceed_max_lines() {
+        let mut ranges = BlameRanges::from_one_based_inclusive_range(1..=5).unwrap();
+        ranges.add_one_based_inclusive_range(6..=10).unwrap();
+
+        assert_eq!(ranges.to_zero_based_exclusive_ranges(7), vec![0..7]);
+    }
+
+    #[test]
+    fn to_zero_based_exclusive_merged_ranges_dont_exceed_max_lines() {
+        let mut ranges = BlameRanges::from_one_based_inclusive_range(1..=4).unwrap();
+        ranges.add_one_based_inclusive_range(6..=10).unwrap();
+
+        assert_eq!(ranges.to_zero_based_exclusive_ranges(7), vec![0..4, 5..7]);
+    }
+
     #[test]
     fn default_is_full_file() {
         let ranges = BlameRanges::default();

gix-blame/src/types.rs

@@ -132,7 +132,20 @@ impl BlameRanges {
                 let full_range = 0..max_lines;
                 vec![full_range]
             }
-            Self::PartialFile(ranges) => ranges.clone(),
+            Self::PartialFile(ranges) => ranges
+                .iter()
+                .filter_map(|range| {
+                    if range.end < max_lines {
+                        return Some(range.clone());
+                    }
+
+                    if range.start < max_lines {
+                        Some(range.start..max_lines)
+                    } else {
+                        None
+                    }
+                })
+                .collect(),
         }
     }
 }