Add BlameEntry::from_unblamed_hunk

Author: cruessler

gix-blame/src/lib.rs

@@ -134,6 +134,14 @@ impl BlameEntry {
             }
         }
     }
+
+    fn from_unblamed_hunk(unblamed_hunk: &UnblamedHunk, commit_id: ObjectId) -> Self {
+        Self {
+            range_in_blamed_file: unblamed_hunk.range_in_blamed_file.clone(),
+            range_in_original_file: unblamed_hunk.range_in_destination.clone(),
+            commit_id,
+        }
+    }
 }
 
 #[derive(Clone, Debug, PartialEq)]
@@ -744,16 +752,11 @@ pub fn blame_file<E>(
             // remaining lines to it, even though we don’t explicitly check whether that is true
             // here. We could perhaps use `needed_to_obtain` to compare `suspect` against an empty
             // tree to validate this assumption.
-            out.extend(hunks_to_blame.iter().map(|hunk| {
-                BlameEntry::new(
-                    hunk.range_in_blamed_file.clone(),
-                    // TODO
-                    // I don’t know whether it is correct to use `range_in_destination` for
-                    // this purpose.
-                    hunk.range_in_destination.clone(),
-                    suspect,
-                )
-            }));
+            out.extend(
+                hunks_to_blame
+                    .iter()
+                    .map(|hunk| BlameEntry::from_unblamed_hunk(hunk, suspect)),
+            );
 
             hunks_to_blame = vec![];
 
@@ -777,16 +780,11 @@ pub fn blame_file<E>(
             gix_diff::tree::recorder::Change::Addition { .. } => {
                 // Every line that has not been blamed yet on a commit, is expected to have been
                 // added when the file was added to the repository.
-                out.extend(hunks_to_blame.iter().map(|hunk| {
-                    BlameEntry::new(
-                        hunk.range_in_blamed_file.clone(),
-                        // TODO
-                        // I don’t know whether it is correct to use `range_in_destination` for
-                        // this purpose.
-                        hunk.range_in_destination.clone(),
-                        suspect,
-                    )
-                }));
+                out.extend(
+                    hunks_to_blame
+                        .iter()
+                        .map(|hunk| BlameEntry::from_unblamed_hunk(hunk, suspect)),
+                );
 
                 hunks_to_blame = vec![];