Commit Diff

Commit:
06ca4d091887ecc59f63c7fa459c57e58122d31b
From:
Stefan Sperling <stsp@stsp.name>
Date:
Message:
stop history traversal once blame has annotated all lines
Actions:
Patch | Tree
commit - 6fcac4576a437f60e050eec706040bdb272eb058
commit + 06ca4d091887ecc59f63c7fa459c57e58122d31b
blob - e32dd4a6f55ae4f90e82a25f0190eb98da4c65ad
blob + 977d7f578441bc9814edefb94eb8001344b09deb
--- lib/blame.c
+++ lib/blame.c
@@ -53,6 +53,7 @@ SLIST_HEAD(got_blame_diff_offsets_list, got_blame_diff
 struct got_blame {
 	FILE *f;
 	int nlines;
+	int nannotated;
 	struct got_blame_line *lines; /* one per line */
 	int ncommits;
 	struct got_blame_diff_offsets_list diff_offsets_list;
@@ -111,6 +112,7 @@ annotate_line(struct got_blame *blame, int lineno, str
 
 	memcpy(&line->id, id, sizeof(line->id));
 	line->annotated = 1;
+	blame->nannotated++;
 	if (cb)
 		err = cb(arg, blame->nlines, lineno, id);
 	return err;
@@ -151,6 +153,8 @@ blame_changes(struct got_blame *blame, struct got_diff
 			    commit_id, cb, arg);
 			if (err)
 				return err;
+			if (blame->nlines == blame->nannotated)
+				return NULL;
 		}
 	}
 
@@ -369,11 +373,13 @@ blame_open(struct got_blame **blamep, const char *path
 					err = NULL;
 				break;
 			}
+			if (blame->nannotated == blame->nlines)
+				break;
 		}
 		id = next_id;
 	}
 
-	if (id) {
+	if (id && blame->nannotated < blame->nlines) {
 		/* Annotate remaining non-annotated lines with last commit. */
 		for (lineno = 1; lineno <= blame->nlines; lineno++) {
 			err = annotate_line(blame, lineno, id, cb, arg);