diff options
author | Junio C Hamano <gitster@pobox.com> | 2016-09-19 13:51:43 -0700 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2016-09-19 13:51:43 -0700 |
commit | c3befaeab9ebd0def9f03d434a52cf40614c65f9 (patch) | |
tree | eaa41d9b527b3bdb4c8a6a78443a9277e1c5740f /hex.c | |
parent | 815a73f7143214510fae6e8a2f2077b2e8694509 (diff) | |
parent | d23309733a5b2a9e1adc304ee50c5a5ed7a087c2 (diff) | |
download | git-c3befaeab9ebd0def9f03d434a52cf40614c65f9.tar.gz |
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.c | 12 |
1 files changed, 2 insertions, 10 deletions
@@ -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; |