On Sun, Jul 8, 2018 at 4:39 PM brian m. carlson <sandals@xxxxxxxxxxxxxxxxxxxx> wrote: > static inline int oidcmp(const struct object_id *oid1, const struct object_id *oid2) > { > - return hashcmp(oid1->hash, oid2->hash); > + return memcmp(oid1->hash, oid2->hash, the_hash_algo->rawsz); > } > Just curious, what's the reasoning for not using the hashcmp anymore? Thanks, Jake