From: Karthik Nayak <karthik.188@gmail.com>
Date: Wed, 20 Aug 2025 13:51:54 GMT
Subject: Re: [PATCH 14/16] packfile: remove `get_packed_git()`
Message-ID: <CAOLa=ZSzgZdqB2OuS=Vi7M-ZCHDzjBCpCdW99co+xdPAB0HMHw@mail.gmail.com>
In-Reply-To: <20250819-b4-pks-packfiles-store-v1-14-1660842e125a@pks.im>

Patrick Steinhardt <ps@pks.im> writes:

[snip]

> diff --git a/builtin/gc.c b/builtin/gc.c
> index 1d30d1af2c..565afda51f 100644
> --- a/builtin/gc.c
> +++ b/builtin/gc.c
> @@ -1422,7 +1422,7 @@ static int incremental_repack_auto_condition(struct gc_config *cfg UNUSED)
>  	if (incremental_repack_auto_limit < 0)
>  		return 1;
>
> -	for (p = get_packed_git(the_repository);
> +	for (p = get_all_packs(the_repository);
>  	     count < incremental_repack_auto_limit && p;
>  	     p = p->next) {
>  		if (!p->multi_pack_index)

Nit: We can get rid of the curly braces here

[snip]