[PATCH v2 05/49] builtin/repack.c: avoid "the_repository" when repacking promisor objects
- From
Taylor Blau <me@ttaylorr.com>
- Date
- Oct 15, 2025, 22:27 UTC
- Message-ID
- <0914ca113ff179bbf824c026fe1c0463a5acb7f1.1760567210.git.me@ttaylorr.com>
- In-Reply-To
- <cover.1760567210.git.me@ttaylorr.com>
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