path: root/builtin/receive-pack.c
diff options
authorJeff King <>2017-02-08 20:52:57 (GMT)
committerJunio C Hamano <>2017-02-08 23:39:55 (GMT)
commit2429d63a46b141bb5006b8c1ea82e2d0163ab626 (patch)
treef5329a7ce5f3ed9671173f201e27b37c39575358 /builtin/receive-pack.c
parent5e8c968c6465d35c9047ab3ed522cb08d46386f5 (diff)
for_each_alternate_ref: pass name/oid instead of ref struct
Breaking down the fields in the interface makes it easier to change the backend of for_each_alternate_ref to something that doesn't use "struct ref" internally. The only field that callers actually look at is the oid, anyway. The refname is kept in the interface as a plausible thing for future code to want. Signed-off-by: Jeff King <> Signed-off-by: Junio C Hamano <>
Diffstat (limited to 'builtin/receive-pack.c')
1 files changed, 4 insertions, 2 deletions
diff --git a/builtin/receive-pack.c b/builtin/receive-pack.c
index 1dbb8a0..d21332d 100644
--- a/builtin/receive-pack.c
+++ b/builtin/receive-pack.c
@@ -277,10 +277,12 @@ static int show_one_alternate_sha1(const unsigned char sha1[20], void *unused)
return 0;
-static void collect_one_alternate_ref(const struct ref *ref, void *data)
+static void collect_one_alternate_ref(const char *refname,
+ const struct object_id *oid,
+ void *data)
struct sha1_array *sa = data;
- sha1_array_append(sa, ref->old_oid.hash);
+ sha1_array_append(sa, oid->hash);
static void write_head_info(void)