refactor

- fix probably critical, and untested error related to the last hunk

Author: Byron

gix-blame/src/file/function.rs

@@ -1,8 +1,5 @@
 use std::num::NonZeroU32;
 
-#[cfg(feature = "blob-experimental")]
-use gix_diff::blob::v2::Hunk;
-
 use gix_diff::{blob::intern::TokenSource, tree::Visit};
 use gix_hash::ObjectId;
 use gix_object::{
@@ -762,6 +759,8 @@ fn blob_changes(
     diff_algorithm: gix_diff::blob::Algorithm,
     stats: &mut Statistics,
 ) -> Result<Vec<Change>, Error> {
+    use std::ops::Range;
+
     /// Record all [`Change`]s to learn about additions, deletions and unchanged portions of a *Source File*.
     struct ChangeRecorder {
         last_seen_after_end: u32,
@@ -781,8 +780,6 @@ fn blob_changes(
         }
     }
 
-    use std::ops::Range;
-
     impl gix_diff::blob::Sink for ChangeRecorder {
         type Out = Vec<Change>;
 
@@ -857,6 +854,8 @@ fn blob_changes(
     diff_algorithm: gix_diff::blob::Algorithm,
     stats: &mut Statistics,
 ) -> Result<Vec<Change>, Error> {
+    use gix_diff::blob::v2::Hunk;
+
     resource_cache.set_resource(
         previous_oid,
         gix_object::tree::EntryKind::Blob,
@@ -886,36 +885,40 @@ fn blob_changes(
     let mut diff = gix_diff::blob::v2::Diff::compute(diff_algorithm, &input);
     diff.postprocess_lines(&input);
 
-    let changes = diff
-        .hunks()
-        .fold((Vec::new(), 0), |(mut hunks, mut last_seen_after_end), hunk| {
-            let Hunk { before, after } = hunk;
+    let mut last_seen_after_end = 0;
+    let mut changes = diff.hunks().fold(Vec::new(), |mut hunks, hunk| {
+        let Hunk { before, after } = hunk;
 
-            // This checks for unchanged hunks.
-            if after.start > last_seen_after_end {
-                hunks.push(Change::Unchanged(last_seen_after_end..after.start));
-            }
+        // This checks for unchanged hunks.
+        if after.start > last_seen_after_end {
+            hunks.push(Change::Unchanged(last_seen_after_end..after.start));
+        }
 
-            match (!before.is_empty(), !after.is_empty()) {
-                (_, true) => {
-                    hunks.push(Change::AddedOrReplaced(
-                        after.start..after.end,
-                        before.end - before.start,
-                    ));
-                }
-                (true, false) => {
-                    hunks.push(Change::Deleted(after.start, before.end - before.start));
-                }
-                (false, false) => unreachable!("BUG: imara-diff provided a non-change"),
+        match (!before.is_empty(), !after.is_empty()) {
+            (_, true) => {
+                hunks.push(Change::AddedOrReplaced(
+                    after.start..after.end,
+                    before.end - before.start,
+                ));
+            }
+            (true, false) => {
+                hunks.push(Change::Deleted(after.start, before.end - before.start));
             }
+            (false, false) => unreachable!("BUG: imara-diff provided a non-change"),
+        }
 
-            last_seen_after_end = after.end;
+        last_seen_after_end = after.end;
 
-            (hunks, last_seen_after_end)
-        });
+        hunks
+    });
+
+    let total_number_of_lines = input.after.len() as u32;
+    if input.after.len() > last_seen_after_end as usize {
+        changes.push(Change::Unchanged(last_seen_after_end..total_number_of_lines));
+    }
 
     stats.blobs_diffed += 1;
-    Ok(changes.0)
+    Ok(changes)
 }
 
 fn find_path_entry_in_commit(

gix-blame/tests/blame.rs

@@ -303,7 +303,7 @@ fn diff_disparity_imara_diff_v1() {
     diff_disparity_base();
 }
 
-/// As of 2025-12-07, both tests are expected to pass. They use `imara-diff` 0.2 under the hood.
+/// As of 2025-12-07, both algorithms are expected to pass. They use `imara-diff` 0.2 under the hood.
 ///
 /// Context: https://github.com/Byron/gitoxide/pull/1453#issuecomment-2371013904
 #[test]