diff options
author | Patrick Steinhardt <ps@pks.im> | 2019-01-23 10:42:46 +0100 |
---|---|---|
committer | Patrick Steinhardt <ps@pks.im> | 2019-02-15 13:16:48 +0100 |
commit | 351eeff3b2a666b8ead5302c1130629597438df6 (patch) | |
tree | 5b494d3e5912949980fbbbb4945da4c2eb630aa1 /src/pack.c | |
parent | bda0839734bad8351e1dbc9c7beb8ae1f00d831e (diff) | |
download | libgit2-351eeff3b2a666b8ead5302c1130629597438df6.tar.gz |
maps: use uniform lifecycle management functions
Currently, the lifecycle functions for maps (allocation, deallocation, resize)
are not named in a uniform way and do not have a uniform function signature.
Rename the functions to fix that, and stick to libgit2's naming scheme of saying
`git_foo_new`. This results in the following new interface for allocation:
- `int git_<t>map_new(git_<t>map **out)` to allocate a new map, returning an
error code if we ran out of memory
- `void git_<t>map_free(git_<t>map *map)` to free a map
- `void git_<t>map_clear(git<t>map *map)` to remove all entries from a map
This commit also fixes all existing callers.
Diffstat (limited to 'src/pack.c')
-rw-r--r-- | src/pack.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/pack.c b/src/pack.c index cdd7e9e60..88af6afd4 100644 --- a/src/pack.c +++ b/src/pack.c @@ -92,8 +92,8 @@ static void cache_free(git_pack_cache *cache) static int cache_init(git_pack_cache *cache) { - cache->entries = git_offmap_alloc(); - GIT_ERROR_CHECK_ALLOC(cache->entries); + if (git_offmap_new(&cache->entries) < 0) + return -1; cache->memory_limit = GIT_PACK_CACHE_MEMORY_LIMIT; |