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/api/vim.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/api/vim.c')
| -rw-r--r-- | src/nvim/api/vim.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/nvim/api/vim.c b/src/nvim/api/vim.c index 5aed85ef08..90c43a1b04 100644 --- a/src/nvim/api/vim.c +++ b/src/nvim/api/vim.c @@ -63,7 +63,7 @@ void api_vim_free_all_mem(void) { String name; handle_T id; - map_foreach((&namespace_ids), name, id, { + map_foreach(&namespace_ids, name, id, { (void)id; xfree(name.data); }) @@ -1584,7 +1584,7 @@ Dictionary nvim_get_namespaces(void) String name; handle_T id; - map_foreach((&namespace_ids), name, id, { + map_foreach(&namespace_ids, name, id, { PUT(retval, name.data, INTEGER_OBJ(id)); }) |