From: Junio C Hamano Date: Mon, 10 Oct 2016 22:48:33 GMT Subject: Re: [PATCH v2 2/3] serialize collection of refs that contain submodule changes Message-ID: In-Reply-To: <81bdbf6a1295c17c1b9233c91da6e5eb4583785e.1475851621.git.hvoigt@hvoigt.net> Heiko Voigt writes: > +static int append_hash_to_argv(const unsigned char sha1[20], void *data) > +{ > + struct argv_array *argv = (struct argv_array *) data; > + argv_array_push(argv, sha1_to_hex(sha1)); > + return 0; > +} Do we have struct object_id readily available in the caller? ... goes and looks ... No, this is part of sha1_array API, so this callback is perfectly fine.