summaryrefslogtreecommitdiff
path: root/hex.c
diff options
context:
space:
mode:
authorJunio C Hamano <gitster@pobox.com>2016-09-19 20:51:43 (GMT)
committerJunio C Hamano <gitster@pobox.com>2016-09-19 20:51:43 (GMT)
commitc3befaeab9ebd0def9f03d434a52cf40614c65f9 (patch)
treeeaa41d9b527b3bdb4c8a6a78443a9277e1c5740f /hex.c
parent815a73f7143214510fae6e8a2f2077b2e8694509 (diff)
parentd23309733a5b2a9e1adc304ee50c5a5ed7a087c2 (diff)
downloadgit-c3befaeab9ebd0def9f03d434a52cf40614c65f9.zip
git-c3befaeab9ebd0def9f03d434a52cf40614c65f9.tar.gz
git-c3befaeab9ebd0def9f03d434a52cf40614c65f9.tar.bz2
Merge branch 'rs/hex2chr' into maint
Code cleanup. * rs/hex2chr: introduce hex2chr() for converting two hexadecimal digits to a character
Diffstat (limited to 'hex.c')
-rw-r--r--hex.c12
1 files changed, 2 insertions, 10 deletions
diff --git a/hex.c b/hex.c
index 9619b67..ab2610e 100644
--- a/hex.c
+++ b/hex.c
@@ -39,16 +39,8 @@ int get_sha1_hex(const char *hex, unsigned char *sha1)
{
int i;
for (i = 0; i < GIT_SHA1_RAWSZ; i++) {
- unsigned int val;
- /*
- * hex[1]=='\0' is caught when val is checked below,
- * but if hex[0] is NUL we have to avoid reading
- * past the end of the string:
- */
- if (!hex[0])
- return -1;
- val = (hexval(hex[0]) << 4) | hexval(hex[1]);
- if (val & ~0xff)
+ int val = hex2chr(hex);
+ if (val < 0)
return -1;
*sha1++ = val;
hex += 2;