Skip to content

Commit 6f38adf

Browse files
telezhnayagitster
authored andcommitted
ref-filter: merge get_obj and get_object
Inline get_obj(): it would be easier to edit the code without this split. Signed-off-by: Olga Telezhnaia <[email protected]> Signed-off-by: Junio C Hamano <[email protected]>
1 parent 7085839 commit 6f38adf

File tree

1 file changed

+11
-25
lines changed

1 file changed

+11
-25
lines changed

ref-filter.c

Lines changed: 11 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -797,24 +797,6 @@ int verify_ref_format(struct ref_format *format)
797797
return 0;
798798
}
799799

800-
/*
801-
* Given an object name, read the object data and size, and return a
802-
* "struct object". If the object data we are returning is also borrowed
803-
* by the "struct object" representation, set *eaten as well---it is a
804-
* signal from parse_object_buffer to us not to free the buffer.
805-
*/
806-
static void *get_obj(const struct object_id *oid, struct object **obj, unsigned long *sz, int *eaten)
807-
{
808-
enum object_type type;
809-
void *buf = read_object_file(oid, &type, sz);
810-
811-
if (buf)
812-
*obj = parse_object_buffer(oid, type, *sz, buf, eaten);
813-
else
814-
*obj = NULL;
815-
return buf;
816-
}
817-
818800
static int grab_objectname(const char *name, const struct object_id *oid,
819801
struct atom_value *v, struct used_atom *atom)
820802
{
@@ -1437,20 +1419,24 @@ static const char *get_refname(struct used_atom *atom, struct ref_array_item *re
14371419
}
14381420

14391421
static int get_object(struct ref_array_item *ref, const struct object_id *oid,
1440-
int deref, struct object **obj, struct strbuf *err)
1422+
int deref, struct object **obj, struct strbuf *err)
14411423
{
14421424
int eaten;
14431425
int ret = 0;
14441426
unsigned long size;
1445-
void *buf = get_obj(oid, obj, &size, &eaten);
1427+
enum object_type type;
1428+
void *buf = read_object_file(oid, &type, &size);
14461429
if (!buf)
14471430
ret = strbuf_addf_ret(err, -1, _("missing object %s for %s"),
14481431
oid_to_hex(oid), ref->refname);
1449-
else if (!*obj)
1450-
ret = strbuf_addf_ret(err, -1, _("parse_object_buffer failed on %s for %s"),
1451-
oid_to_hex(oid), ref->refname);
1452-
else
1453-
grab_values(ref->value, deref, *obj, buf, size);
1432+
else {
1433+
*obj = parse_object_buffer(oid, type, size, buf, &eaten);
1434+
if (!*obj)
1435+
ret = strbuf_addf_ret(err, -1, _("parse_object_buffer failed on %s for %s"),
1436+
oid_to_hex(oid), ref->refname);
1437+
else
1438+
grab_values(ref->value, deref, *obj, buf, size);
1439+
}
14541440
if (!eaten)
14551441
free(buf);
14561442
return ret;

0 commit comments

Comments
 (0)