diff options
| author | Taylor Blau <me@ttaylorr.com> | 2022-09-20 03:55:53 +0200 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2022-09-21 19:21:47 +0200 |
| commit | 0a8e5614924abaac17008aea3cef2ee8fa25f26a (patch) | |
| tree | 5d0a5d240965d8da43e45f160d8d6fcde98df8b6 /midx.c | |
| parent | midx.c: replace `xcalloc()` with `CALLOC_ARRAY()` (diff) | |
| download | git-0a8e5614924abaac17008aea3cef2ee8fa25f26a.tar.xz git-0a8e5614924abaac17008aea3cef2ee8fa25f26a.zip | |
midx.c: remove unnecessary loop condition
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>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'midx.c')
| -rw-r--r-- | midx.c | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -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++; } |