noted a rare bug

2bndy5 · 2bndy5 · commit 8327dae73a5c · 2025-01-03T13:29:05.000-08:00
diff --git a/cpp-linter/src/git.rs b/cpp-linter/src/git.rs
@@ -64,6 +64,10 @@ pub fn get_diff(repo: &Repository) -> Result<git2::Diff> {
         }
     }
 
+    // RARE BUG when `head` is the first commit in the repo! Affects local-only runs.
+    // > panicked at cpp-linter\src\git.rs:73:43:
+    // > called `Result::unwrap()` on an `Err` value:
+    // > Error { code: -3, class: 3, message: "parent 0 does not exist" }
     if has_staged_files {
         // get diff for staged files only
         repo.diff_tree_to_index(Some(&head), None, None)

Original file line number	Diff line number	Diff line change
`@@ -64,6 +64,10 @@ pub fn get_diff(repo: &Repository) -> Result<git2::Diff> {`
`64`	`64`	`}`
`65`	`65`	`}`
`66`	`66`
	`67`	+ // RARE BUG when `head` is the first commit in the repo! Affects local-only runs.
	`68`	`+ // > panicked at cpp-linter\src\git.rs:73:43:`
	`69`	+ // > called `Result::unwrap()` on an `Err` value:
	`70`	`+ // > Error { code: -3, class: 3, message: "parent 0 does not exist" }`
`67`	`71`	`if has_staged_files {`
`68`	`72`	`// get diff for staged files only`
`69`	`73`	`repo.diff_tree_to_index(Some(&head), None, None)`