The verify-pack command walks objects in a pack for contents verification in a very naive way, namely, in the object name order. It is inefficient compared to the way index-pack does, which verifies contents of the objects in the order delta-chain is applied. As the result, the former command would end up reconstituting the same intermediate object in a delta chain number of times while the latter would recreate an object only once. This series, which is still a WIP, is to teach index-pack "--verify" option, in order to eventually replace verify-pack with it. The basic idea, which comes from Shawn Pearce, is to run index-pack on a packfile being verified, and compare the resulting .idx file with the existing one, and make sure they match. Junio C Hamano (5): index-pack: group the delta-base array entries also by type write_idx_file: introduce a struct to hold idx customization options index-pack: --verify write_idx_file: need_large_offset() helper function index-pack --verify: read anomalous offsets from v2 idx file builtin/index-pack.c | 178 ++++++++++++++++++++++++++++++++++++++---------- builtin/pack-objects.c | 20 +++--- csum-file.c | 46 ++++++++++++- csum-file.h | 2 + fast-import.c | 10 ++- pack-write.c | 82 ++++++++++++++++------ pack.h | 23 +++++- t/t5302-pack-index.sh | 18 +++++ 8 files changed, 301 insertions(+), 78 deletions(-) -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html