On Thu, Jan 28, 2021 at 05:48:22PM -0500, Taylor Blau wrote:
> > builtin/name-rev.c | 8 ++++----
> > commit-graph.c | 28 ++++++++++++++--------------
> > commit.c | 10 +++++-----
> > hash-lookup.c | 18 +++++++++---------
> > hash-lookup.h | 10 +++++-----
>
> I wondered briefly if we should rename this to oid-lookup.{c,h}, but I
> think the answer is "no", since we still have bsearch_hash() in this
> header.
>
> Probably a single header with "hash" in the name is better than two
> headers each containing a single function (and one containing an
> additional typedef).
>
> So, I think that what you did here is good.
Yeah. I'd be more inclined to do so if renames weren't still a minor
pain when following history, etc. Getting rid of "sha1" crosses my bar
of worth doing. The distinction between "hash" and "oid" here doesn't.
(plus as you note, we'd really have to split it, which is even more
annoying).
-Peff
