Skip to content

Commit 0a8e561

Browse files
ttaylorrgitster
authored andcommitted
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 <[email protected]> Signed-off-by: Junio C Hamano <[email protected]>
1 parent cb6c48c commit 0a8e561

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

midx.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1928,7 +1928,7 @@ static int fill_included_packs_batch(struct repository *r,
19281928
pack_info[i].mtime = m->packs[i]->mtime;
19291929
}
19301930

1931-
for (i = 0; batch_size && i < m->num_objects; i++) {
1931+
for (i = 0; i < m->num_objects; i++) {
19321932
uint32_t pack_int_id = nth_midxed_pack_int_id(m, i);
19331933
pack_info[pack_int_id].referenced_objects++;
19341934
}

0 commit comments

Comments
 (0)