summaryrefslogtreecommitdiff
path: root/hex.c
diff options
context:
space:
mode:
authorLibravatar Junio C Hamano <gitster@pobox.com>2016-11-29 13:27:55 -0800
committerLibravatar Junio C Hamano <gitster@pobox.com>2016-11-29 13:27:55 -0800
commitf2ad912f996d2f21a8e6bccc10c2d3ef7acbf3ea (patch)
treeddd478a6e752b491093c37921a5848d38dd32e39 /hex.c
parentMerge branch 'mm/send-email-cc-cruft-after-address' into maint (diff)
parenthex: make wraparound of the index into ring-buffer explicit (diff)
downloadtgif-f2ad912f996d2f21a8e6bccc10c2d3ef7acbf3ea.tar.xz
Merge branch 'rs/ring-buffer-wraparound' into maint
The code that we have used for the past 10+ years to cycle 4-element ring buffers turns out to be not quite portable in theoretical world. * rs/ring-buffer-wraparound: hex: make wraparound of the index into ring-buffer explicit
Diffstat (limited to 'hex.c')
-rw-r--r--hex.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/hex.c b/hex.c
index ab2610e498..845b01a874 100644
--- a/hex.c
+++ b/hex.c
@@ -78,7 +78,8 @@ char *sha1_to_hex(const unsigned char *sha1)
{
static int bufno;
static char hexbuffer[4][GIT_SHA1_HEXSZ + 1];
- return sha1_to_hex_r(hexbuffer[3 & ++bufno], sha1);
+ bufno = (bufno + 1) % ARRAY_SIZE(hexbuffer);
+ return sha1_to_hex_r(hexbuffer[bufno], sha1);
}
char *oid_to_hex(const struct object_id *oid)