threads / patch / 44185

patchblame: use DEFAULT_ABBREV macro

Subject: [PATCH] blame: use DEFAULT_ABBREV macro

## tl;dr

One message between Sep 28, 2016 and Sep 28, 2016. Diffs are folded; open one to read it.

replies: 0people: 1as markdown or json

Junio C Hamano· Sep 28, 2016, 21:57 UTC · lore

This does not make any practical difference in today's code, but everybody else accesses the default abbreviation length via the DEFAULT_ABBREV macro. Make sure this oddball codepath does not stray from the convention.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
---
 builtin/blame.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
Show changes to builtin/blame.c +1 −1
diff --git a/builtin/blame.c b/builtin/blame.c
index a5bbf91..f618392 100644
--- a/builtin/blame.c
+++ b/builtin/blame.c
@@ -2111,7 +2111,7 @@ static void find_alignment(struct scoreboard *sb, int *option)
 	unsigned largest_score = 0;
 	struct blame_entry *e;
 	int compute_auto_abbrev = (abbrev < 0);
-	int auto_abbrev = default_abbrev;
+	int auto_abbrev = DEFAULT_ABBREV;
 
 	for (e = sb->ent; e; e = e->next) {
 		struct origin *suspect = e->suspect;
-- 
2.10.0-582-gfa57e49

← back to recent threads