diff options
| author | Taylor Blau <me@ttaylorr.com> | 2026-02-24 14:00:26 -0500 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2026-02-24 11:16:34 -0800 |
| commit | 93c67df7511057e7c8fda169671a44322c04c2d0 (patch) | |
| tree | db23b3e288e82f16389f7175a9fbefe1896ab8ab /midx-write.c | |
| parent | 5f3e7f7279da72b20f99a2bcd26d146f7edaef9d (diff) | |
| download | git-93c67df7511057e7c8fda169671a44322c04c2d0.tar.xz | |
midx-write.c: enumerate `pack_int_id` values directly
Our `midx-write.c::fill_packs_from_midx()` function currently enumerates
the range [0, m->num_packs), and then shifts its index variable up by
`m->num_packs_in_base` to produce a valid `pack_int_id`.
Instead, directly enumerate the range:
[m->num_packs_in_base, m->num_packs_in_base + m->num_packs)
, which are the original pack_int_ids themselves as opposed to the
indexes of those packs relative to the MIDX layer they are contained
within.
Signed-off-by: Taylor Blau <me@ttaylorr.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'midx-write.c')
| -rw-r--r-- | midx-write.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/midx-write.c b/midx-write.c index c54113cdc8..80334914d3 100644 --- a/midx-write.c +++ b/midx-write.c @@ -935,11 +935,11 @@ static int fill_packs_from_midx(struct write_midx_context *ctx) for (m = ctx->m; m; m = m->base_midx) { uint32_t i; - for (i = 0; i < m->num_packs; i++) { + for (i = m->num_packs_in_base; + i < m->num_packs_in_base + m->num_packs; i++) { ALLOC_GROW(ctx->info, ctx->nr + 1, ctx->alloc); - if (fill_pack_from_midx(&ctx->info[ctx->nr], m, - m->num_packs_in_base + i) < 0) + if (fill_pack_from_midx(&ctx->info[ctx->nr], m, i) < 0) return -1; ctx->nr++; |