Thread (13 messages) flat view 13 messages, 4 authors, 2017-10-04

Re: [PATCH v2] oidmap: map with OID as key

From: Jeff King <hidden>
Date: 2017-10-04 07:46:07

On Tue, Oct 03, 2017 at 05:29:01PM -0700, Jonathan Tan wrote:
At this point I decided that I prefer the thin wrapper, but the "light
touch" (struct oidmap_entry, oidcmpfn(), oidmap_get() only) still
better than the status quo.
OK. I can certainly live with that. And worst case, I suppose, is that a
caller wants some underlying hashmap function and we just have to extend
the oidmap API to include it. It's not like we're adding new hashmap
functions willy-nilly.

-Peff
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help