[PATCH 6/7] midx.c: remove unnecessary loop condition
- From
Taylor Blau <me@ttaylorr.com>
- Date
- Sep 20, 2022, 01:55 UTC
- Message-ID
- <4c9c18de2d3a36c0939d21027b73762b20298477.1663638929.git.me@ttaylorr.com>
- In-Reply-To
- <cover.1663638929.git.me@ttaylorr.com>
The fill_included_packs_batch() routine is responsible for aggregating objects in packs with a non-zero value for the `--batch-size` option of the `git multi-pack-index repack` sub-command.
Since this routine is explicitly called only when `--batch-size` is non-zero, there is no point in checking that this is the case in our loop condition.
Remove the unnecessary part of this condition to avoid confusion.
Signed-off-by: Taylor Blau <me@ttaylorr.com> --- midx.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/midx.c b/midx.c index d703fc5a16..273704caed 100644 --- a/midx.c +++ b/midx.c @@ -1928,7 +1928,7 @@ static int fill_included_packs_batch(struct repository *r, pack_info[i].mtime = m->packs[i]->mtime; } - for (i = 0; batch_size && i < m->num_objects; i++) { + for (i = 0; i < m->num_objects; i++) { uint32_t pack_int_id = nth_midxed_pack_int_id(m, i); pack_info[pack_int_id].referenced_objects++; }
-- 2.37.0.1.g1379af2e9d