oidmap: map with OID as key
This is similar to using the hashmap in hashmap.c, but with an easier-to-use API. In particular, custom entry comparisons no longer need to be written, and lookups can be done without constructing a temporary entry structure. This is implemented as a thin wrapper over the hashmap API. In particular, this means that there is an additional 4-byte overhead due to the fact that the first 4 bytes of the hash is redundantly stored. For now, I'm taking the simpler approach, but if need be, we can reimplement oidmap without affecting the callers significantly. oidset has been updated to use oidmap. Signed-off-by: Jonathan Tan <jonathantanmy@google.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
This commit is contained in:
committed by
Junio C Hamano
parent
ea220ee40c
commit
9e6fabde82
6
oidset.h
6
oidset.h
@ -1,6 +1,8 @@
|
||||
#ifndef OIDSET_H
|
||||
#define OIDSET_H
|
||||
|
||||
#include "oidmap.h"
|
||||
|
||||
/**
|
||||
* This API is similar to sha1-array, in that it maintains a set of object ids
|
||||
* in a memory-efficient way. The major differences are:
|
||||
@ -17,10 +19,10 @@
|
||||
* A single oidset; should be zero-initialized (or use OIDSET_INIT).
|
||||
*/
|
||||
struct oidset {
|
||||
struct hashmap map;
|
||||
struct oidmap map;
|
||||
};
|
||||
|
||||
#define OIDSET_INIT { { NULL } }
|
||||
#define OIDSET_INIT { OIDMAP_INIT }
|
||||
|
||||
/**
|
||||
* Returns true iff `set` contains `oid`.
|
||||
|
||||
Reference in New Issue
Block a user