Thanks to Jonathan and Brian for the help with the proper way to handle OIDs and existing callers to bsearch_hash(). This patch includes one commit that Brian sent in the previous discussion (included again here for completeness). Derrick Stolee (2): packfile: define and use bsearch_pack() sha1_name: use bsearch_pack() for abbreviations brian m. carlson (1): sha1_name: use bsearch_hash() for abbreviations packfile.c | 42 ++++++++++++++++++++++++++---------------- packfile.h | 8 ++++++++ sha1_name.c | 28 ++++++---------------------- 3 files changed, 40 insertions(+), 38 deletions(-) base-commit: 1a750441a7360b29fff7a414649ece1d35acaca6 -- 2.17.0.rc0