Updates from v1: - use uint32_t so that we can operate on packfiles of up to 4G objects (this also means that I had to change the signature of the function) - don't hide types Derrick: you'll need to slightly change your patch to use the new API. As for find_abbrev_len_for_pack(), that's a good idea - I didn't do it in this set but it definitely should be done. Jonathan Tan (2): packfile: remove GIT_DEBUG_LOOKUP log statements packfile: refactor hash search with fanout table packfile.c | 29 ++++------------------------- sha1-lookup.c | 28 ++++++++++++++++++++++++++++ sha1-lookup.h | 22 ++++++++++++++++++++++ 3 files changed, 54 insertions(+), 25 deletions(-) -- 2.16.0.rc1.238.g530d649a79-goog