mirror of
https://github.com/git/git
synced 2024-08-24 18:26:02 +00:00
sha1_object_info_extended: rename "status" to "type"
The value we get from each low-level object_info function (e.g., loose, packed) is actually the object type (or -1 for error). Let's explicitly call it "type", which will make further refactorings easier to read. Signed-off-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com>
This commit is contained in:
parent
25fba78d36
commit
f2f57e31f6
20
sha1_file.c
20
sha1_file.c
|
@ -2375,7 +2375,7 @@ int sha1_object_info_extended(const unsigned char *sha1, struct object_info *oi)
|
||||||
{
|
{
|
||||||
struct cached_object *co;
|
struct cached_object *co;
|
||||||
struct pack_entry e;
|
struct pack_entry e;
|
||||||
int status, rtype;
|
int type, rtype;
|
||||||
|
|
||||||
co = find_cached_object(sha1);
|
co = find_cached_object(sha1);
|
||||||
if (co) {
|
if (co) {
|
||||||
|
@ -2389,23 +2389,23 @@ int sha1_object_info_extended(const unsigned char *sha1, struct object_info *oi)
|
||||||
|
|
||||||
if (!find_pack_entry(sha1, &e)) {
|
if (!find_pack_entry(sha1, &e)) {
|
||||||
/* Most likely it's a loose object. */
|
/* Most likely it's a loose object. */
|
||||||
status = sha1_loose_object_info(sha1, oi->sizep, oi->disk_sizep);
|
type = sha1_loose_object_info(sha1, oi->sizep, oi->disk_sizep);
|
||||||
if (status >= 0) {
|
if (type >= 0) {
|
||||||
oi->whence = OI_LOOSE;
|
oi->whence = OI_LOOSE;
|
||||||
return status;
|
return type;
|
||||||
}
|
}
|
||||||
|
|
||||||
/* Not a loose object; someone else may have just packed it. */
|
/* Not a loose object; someone else may have just packed it. */
|
||||||
reprepare_packed_git();
|
reprepare_packed_git();
|
||||||
if (!find_pack_entry(sha1, &e))
|
if (!find_pack_entry(sha1, &e))
|
||||||
return status;
|
return type;
|
||||||
}
|
}
|
||||||
|
|
||||||
status = packed_object_info(e.p, e.offset, oi->sizep, &rtype,
|
type = packed_object_info(e.p, e.offset, oi->sizep, &rtype,
|
||||||
oi->disk_sizep);
|
oi->disk_sizep);
|
||||||
if (status < 0) {
|
if (type < 0) {
|
||||||
mark_bad_packed_object(e.p, sha1);
|
mark_bad_packed_object(e.p, sha1);
|
||||||
status = sha1_object_info_extended(sha1, oi);
|
type = sha1_object_info_extended(sha1, oi);
|
||||||
} else if (in_delta_base_cache(e.p, e.offset)) {
|
} else if (in_delta_base_cache(e.p, e.offset)) {
|
||||||
oi->whence = OI_DBCACHED;
|
oi->whence = OI_DBCACHED;
|
||||||
} else {
|
} else {
|
||||||
|
@ -2416,7 +2416,7 @@ int sha1_object_info_extended(const unsigned char *sha1, struct object_info *oi)
|
||||||
rtype == OBJ_OFS_DELTA);
|
rtype == OBJ_OFS_DELTA);
|
||||||
}
|
}
|
||||||
|
|
||||||
return status;
|
return type;
|
||||||
}
|
}
|
||||||
|
|
||||||
int sha1_object_info(const unsigned char *sha1, unsigned long *sizep)
|
int sha1_object_info(const unsigned char *sha1, unsigned long *sizep)
|
||||||
|
|
Loading…
Reference in a new issue