diff options
author | Taylor Blau <me@ttaylorr.com> | 2023-09-13 15:17:51 -0400 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2023-09-13 12:32:47 -0700 |
commit | f2d3bf178aaf4e590f3618d657e14aceb44514f0 (patch) | |
tree | f81570d8f64336b566bb7e20eecb2c162900e929 /commit.c | |
parent | 639c4a3992337e15c0faa2fa6d73462cc76ca7b9 (diff) |
builtin/repack.c: extract redundant pack cleanup for existing packs
To remove redundant packs at the end of a repacking operation, Git uses
its `remove_redundant_pack()` function in a loop over the set of
pre-existing, non-kept packs.
In a later commit, we will split this list into two, one for
pre-existing cruft pack(s), and another for non-cruft pack(s). Prepare
for this by factoring out the routine to loop over and delete redundant
packs into its own function.
Instead of calling `remove_redundant_pack()` directly, we now will call
`remove_redundant_existing_packs()`, which itself dispatches a call to
`remove_redundant_packs_1()`. Note that the geometric repacking code
will still call `remove_redundant_pack()` directly, but see the previous
commit for more details.
Having `remove_redundant_packs_1()` exist as a separate function may
seem like overkill in this patch. However, a later patch will call
`remove_redundant_packs_1()` once over two separate lists, so this
refactoring sets us up for that.
Signed-off-by: Taylor Blau <me@ttaylorr.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'commit.c')
0 files changed, 0 insertions, 0 deletions