Convert memcpy(a,b,20) to hashcpy(a,b).
This abstracts away the size of the hash values when copying them from memory location to memory location, much as the introduction of hashcmp abstracted away hash value comparsion. A few call sites were using char* rather than unsigned char* so I added the cast rather than open hashcpy to be void*. This is a reasonable tradeoff as most call sites already use unsigned char* and the existing hashcmp is also declared to be unsigned char*. [jc: Splitted the patch to "master" part, to be followed by a patch for merge-recursive.c which is not in "master" yet. Fixed the cast in the latter hunk to combine-diff.c which was wrong in the original. Also converted ones left-over in combine-diff.c, diff-lib.c and upload-pack.c ] Signed-off-by: Shawn O. Pearce <spearce@spearce.org> Signed-off-by: Junio C Hamano <junkio@cox.net>
This commit is contained in:

committed by
Junio C Hamano

parent
b05faa2da9
commit
e702496e43
22
sha1_name.c
22
sha1_name.c
@ -103,7 +103,7 @@ static int find_short_packed_object(int len, const unsigned char *match, unsigne
|
||||
!match_sha(len, match, next)) {
|
||||
/* unique within this pack */
|
||||
if (!found) {
|
||||
memcpy(found_sha1, now, 20);
|
||||
hashcpy(found_sha1, now);
|
||||
found++;
|
||||
}
|
||||
else if (hashcmp(found_sha1, now)) {
|
||||
@ -120,7 +120,7 @@ static int find_short_packed_object(int len, const unsigned char *match, unsigne
|
||||
}
|
||||
}
|
||||
if (found == 1)
|
||||
memcpy(sha1, found_sha1, 20);
|
||||
hashcpy(sha1, found_sha1);
|
||||
return found;
|
||||
}
|
||||
|
||||
@ -140,13 +140,13 @@ static int find_unique_short_object(int len, char *canonical,
|
||||
if (1 < has_unpacked || 1 < has_packed)
|
||||
return SHORT_NAME_AMBIGUOUS;
|
||||
if (has_unpacked != has_packed) {
|
||||
memcpy(sha1, (has_packed ? packed_sha1 : unpacked_sha1), 20);
|
||||
hashcpy(sha1, (has_packed ? packed_sha1 : unpacked_sha1));
|
||||
return 0;
|
||||
}
|
||||
/* Both have unique ones -- do they match? */
|
||||
if (hashcmp(packed_sha1, unpacked_sha1))
|
||||
return SHORT_NAME_AMBIGUOUS;
|
||||
memcpy(sha1, packed_sha1, 20);
|
||||
hashcpy(sha1, packed_sha1);
|
||||
return 0;
|
||||
}
|
||||
|
||||
@ -320,13 +320,13 @@ static int get_parent(const char *name, int len,
|
||||
if (parse_commit(commit))
|
||||
return -1;
|
||||
if (!idx) {
|
||||
memcpy(result, commit->object.sha1, 20);
|
||||
hashcpy(result, commit->object.sha1);
|
||||
return 0;
|
||||
}
|
||||
p = commit->parents;
|
||||
while (p) {
|
||||
if (!--idx) {
|
||||
memcpy(result, p->item->object.sha1, 20);
|
||||
hashcpy(result, p->item->object.sha1);
|
||||
return 0;
|
||||
}
|
||||
p = p->next;
|
||||
@ -347,9 +347,9 @@ static int get_nth_ancestor(const char *name, int len,
|
||||
|
||||
if (!commit || parse_commit(commit) || !commit->parents)
|
||||
return -1;
|
||||
memcpy(sha1, commit->parents->item->object.sha1, 20);
|
||||
hashcpy(sha1, commit->parents->item->object.sha1);
|
||||
}
|
||||
memcpy(result, sha1, 20);
|
||||
hashcpy(result, sha1);
|
||||
return 0;
|
||||
}
|
||||
|
||||
@ -401,7 +401,7 @@ static int peel_onion(const char *name, int len, unsigned char *sha1)
|
||||
o = deref_tag(o, name, sp - name - 2);
|
||||
if (!o || (!o->parsed && !parse_object(o->sha1)))
|
||||
return -1;
|
||||
memcpy(sha1, o->sha1, 20);
|
||||
hashcpy(sha1, o->sha1);
|
||||
}
|
||||
else {
|
||||
/* At this point, the syntax look correct, so
|
||||
@ -413,7 +413,7 @@ static int peel_onion(const char *name, int len, unsigned char *sha1)
|
||||
if (!o || (!o->parsed && !parse_object(o->sha1)))
|
||||
return -1;
|
||||
if (o->type == expected_type) {
|
||||
memcpy(sha1, o->sha1, 20);
|
||||
hashcpy(sha1, o->sha1);
|
||||
return 0;
|
||||
}
|
||||
if (o->type == OBJ_TAG)
|
||||
@ -520,7 +520,7 @@ int get_sha1(const char *name, unsigned char *sha1)
|
||||
memcmp(ce->name, cp, namelen))
|
||||
break;
|
||||
if (ce_stage(ce) == stage) {
|
||||
memcpy(sha1, ce->sha1, 20);
|
||||
hashcpy(sha1, ce->sha1);
|
||||
return 0;
|
||||
}
|
||||
pos++;
|
||||
|
Reference in New Issue
Block a user