diff options
| author | Björn Linse <bjorn.linse@gmail.com> | 2021-08-22 16:03:21 +0200 |
|---|---|---|
| committer | Björn Linse <bjorn.linse@gmail.com> | 2021-08-22 16:15:38 +0200 |
| commit | de21e6ef3d9af96d2b71e54d8148d28b5fc9f22e (patch) | |
| tree | dc153c0cf6ac08e6f0f5508dcc2603618f314ef6 /src/nvim/map.c | |
| parent | db1b0ee3b30fd4cd323907c7f24bd575c22e68f0 (diff) | |
| download | rneovim-de21e6ef3d9af96d2b71e54d8148d28b5fc9f22e.tar.gz rneovim-de21e6ef3d9af96d2b71e54d8148d28b5fc9f22e.tar.bz2 rneovim-de21e6ef3d9af96d2b71e54d8148d28b5fc9f22e.zip | |
refactor(map): remove extra-allocating map_new/map_free functions
Note: the reason for removing them is not that there after this refactor
is no use of them, but rather that having them available is an
anti-pattern: they manange an _extra_ heap allocation which has
nothing to do with the functionality of the map itself (khash
manages the real buffers internally). In case there happens to
be a reason to allocate the map structure itself later, this
should be made explicit using xcalloc/xfree calls.
Diffstat (limited to 'src/nvim/map.c')
| -rw-r--r-- | src/nvim/map.c | 13 |
1 files changed, 0 insertions, 13 deletions
diff --git a/src/nvim/map.c b/src/nvim/map.c index d956699b21..ccd332192e 100644 --- a/src/nvim/map.c +++ b/src/nvim/map.c @@ -54,19 +54,6 @@ INITIALIZER_DECLARE(T, U, __VA_ARGS__); \ __KHASH_IMPL(T##_##U##_map,, T, U, 1, T##_hash, T##_eq) \ \ - Map(T, U) *map_##T##_##U##_new() \ - { \ - Map(T, U) *rv = xcalloc(1, sizeof(Map(T, U))); \ - /* khash_t table member is zero-initialized */ \ - return rv; \ - } \ - \ - void map_##T##_##U##_free(Map(T, U) *map) \ - { \ - kh_dealloc(T##_##U##_map, &map->table); \ - xfree(map); \ - } \ - \ void map_##T##_##U##_destroy(Map(T, U) *map) \ { \ kh_dealloc(T##_##U##_map, &map->table); \ |