Skip to content

Commit ab47df2

Browse files
bk2204gitster
authored andcommitted
builtin/merge: switch to use the_hash_algo
Switch uses of GIT_SHA1_HEXSZ to use the_hash_algo instead. Signed-off-by: brian m. carlson <[email protected]> Signed-off-by: Junio C Hamano <[email protected]>
1 parent 5188eb5 commit ab47df2

File tree

1 file changed

+6
-5
lines changed

1 file changed

+6
-5
lines changed

builtin/merge.c

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1034,6 +1034,7 @@ static void handle_fetch_head(struct commit_list **remotes, struct strbuf *merge
10341034
const char *filename;
10351035
int fd, pos, npos;
10361036
struct strbuf fetch_head_file = STRBUF_INIT;
1037+
const unsigned hexsz = the_hash_algo->hexsz;
10371038

10381039
if (!merge_names)
10391040
merge_names = &fetch_head_file;
@@ -1059,16 +1060,16 @@ static void handle_fetch_head(struct commit_list **remotes, struct strbuf *merge
10591060
else
10601061
npos = merge_names->len;
10611062

1062-
if (npos - pos < GIT_SHA1_HEXSZ + 2 ||
1063+
if (npos - pos < hexsz + 2 ||
10631064
get_oid_hex(merge_names->buf + pos, &oid))
10641065
commit = NULL; /* bad */
1065-
else if (memcmp(merge_names->buf + pos + GIT_SHA1_HEXSZ, "\t\t", 2))
1066+
else if (memcmp(merge_names->buf + pos + hexsz, "\t\t", 2))
10661067
continue; /* not-for-merge */
10671068
else {
1068-
char saved = merge_names->buf[pos + GIT_SHA1_HEXSZ];
1069-
merge_names->buf[pos + GIT_SHA1_HEXSZ] = '\0';
1069+
char saved = merge_names->buf[pos + hexsz];
1070+
merge_names->buf[pos + hexsz] = '\0';
10701071
commit = get_merge_parent(merge_names->buf + pos);
1071-
merge_names->buf[pos + GIT_SHA1_HEXSZ] = saved;
1072+
merge_names->buf[pos + hexsz] = saved;
10721073
}
10731074
if (!commit) {
10741075
if (ptr)

0 commit comments

Comments
 (0)