On Mon, Apr 17, 2023 at 04:21:37PM +0000, Derrick Stolee via GitGitGadget wrote: > builtin/fsck.c | 36 +++++++++++++++++++ > pack-bitmap.c | 4 +-- > pack-revindex.c | 43 +++++++++++++++++++++-- > pack-revindex.h | 16 +++++++++ > t/t5325-reverse-index.sh | 74 ++++++++++++++++++++++++++++++++++++++++ > 5 files changed, 169 insertions(+), 4 deletions(-) I gave this a thorough look and it all looks good to me. I left a couple of minor comments throughout, e.g., s/set up/setup, and replacing 'wc -c' with 'test_file_size', but I doubt either of those are a big enough deal to merit a reroll. So I'd be happy to see this start moving forward as-is. I think that the topic it depends on, tb/pack-revindex-on-disk is similarly ready for merging. Thanks, Taylor
