Fix for when diff has "No newline at end of file"

These lines are sort of separate from the diff somehow? But shouldn't be included in the lookup table anyway
ZedThree · Apr 22, 2021 · 3677b43 · 3677b43
1 parent bba1a48
commit 3677b43
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/review.py b/review.py
@@ -34,6 +34,8 @@ def make_file_line_lookup(diff):
         lookup[filename] = {}
         for hunk in file:
             for line in hunk:
+                if line.diff_line_no is None:
+                    continue
                 if not line.is_removed:
                     lookup[filename][line.target_line_no] = (
                         line.diff_line_no - DIFF_HEADER_LINE_LENGTH