[PATCH v2 05/49] builtin/repack.c: avoid "the_repository" when repacking promisor objects

Taylor Blau
2025-10-15T22:27:32Z

Pass a "struct repository" pointer to the 'repack_promisor_objects()'
function to avoid using "the_repository".

Signed-off-by: Taylor Blau <me@ttaylorr.com>
---
builtin/repack.c | 7 ++++---
1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/builtin/repack.c b/builtin/repack.c
index 93802531e1..4f08b57ddb 100644
--- a/builtin/repack.c
+++ b/builtin/repack.c
@@ -407,7 +407,8 @@ static int has_pack_ext(const struct generated_pack_data *data,
BUG("unknown pack extension: '%s'", ext);
}

-static void repack_promisor_objects(const struct pack_objects_args *args,
+static void repack_promisor_objects(struct repository *repo,
+ const struct pack_objects_args *args,
struct string_list *names)
{
struct child_process cmd = CHILD_PROCESS_INIT;
@@ -424,7 +425,7 @@ static void repack_promisor_objects(const struct pack_objects_args *args,
* {type -> existing pack order} ordering when computing deltas instead
* of a {type -> size} ordering, which may produce better deltas.
*/
- for_each_packed_object(the_repository, write_oid, &cmd,
+ for_each_packed_object(repo, write_oid, &cmd,
FOR_EACH_OBJECT_PROMISOR_ONLY);

if (cmd.in == -1) {
@@ -1458,7 +1459,7 @@ int cmd_repack(int argc,
strvec_push(&cmd.args, "--delta-islands");

if (pack_everything & ALL_INTO_ONE) {
- repack_promisor_objects(&po_args, &names);
+ repack_promisor_objects(repo, &po_args, &names);

if (has_existing_non_kept_packs(&existing) &&
delete_redundant &&
--
2.51.0.540.ga7423965ad8