diff options
| author | Jeff King <peff@peff.net> | 2017-01-06 04:18:08 (GMT) |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2017-01-08 03:34:54 (GMT) |
| commit | ed58d8088b570e7629bfc94b87e433f05229ef3c (patch) | |
| tree | eb096878c25f190d13ce80c35f3e03d74c727c12 /t | |
| parent | 91229834c293302c4456e732ddef7ace0df6e471 (diff) | |
| download | git-ed58d8088b570e7629bfc94b87e433f05229ef3c.zip git-ed58d8088b570e7629bfc94b87e433f05229ef3c.tar.gz git-ed58d8088b570e7629bfc94b87e433f05229ef3c.tar.bz2 | |
blame: handle --no-abbrev
You can already ask blame for full sha1s with "-l" or with
"--abbrev=40". But for consistency with other parts of Git,
we should support "--no-abbrev".
Worse, blame already accepts --no-abbrev, but it's totally
broken. When we see --no-abbrev, the abbrev variable is set
to 0, which is then used as a printf precision. For regular
sha1s, that means we print nothing at all (which is very
wrong). For boundary commits we decrement it to "-1", which
printf interprets as "no limit" (which is almost correct,
except it misses the 39-length magic explained in the
previous commit).
Let's detect --no-abbrev and behave as if --abbrev=40 was
given.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't')
| -rwxr-xr-x | t/t8002-blame.sh | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/t/t8002-blame.sh b/t/t8002-blame.sh index c6347ad..380e1c1 100755 --- a/t/t8002-blame.sh +++ b/t/t8002-blame.sh @@ -114,4 +114,8 @@ test_expect_success 'blame --abbrev=40 behaves like -l' ' check_abbrev 39 --abbrev=40 ^HEAD ' +test_expect_success '--no-abbrev works like --abbrev=40' ' + check_abbrev 40 --no-abbrev +' + test_done |