diff options
author | Justin M. Keyes <justinkz@gmail.com> | 2023-12-07 13:01:42 +0100 |
---|---|---|
committer | Justin M. Keyes <justinkz@gmail.com> | 2023-12-07 13:09:31 +0100 |
commit | a16218d4c6773c474011149661470af6e8c90892 (patch) | |
tree | 4d18136e38aaa7e0afed05e8e7a6b3a754e39089 /src/nvim/api/vimscript.c | |
parent | cca6c4c6986abc67cd213ad1d32d329384a57790 (diff) | |
download | rneovim-a16218d4c6773c474011149661470af6e8c90892.tar.gz rneovim-a16218d4c6773c474011149661470af6e8c90892.tar.bz2 rneovim-a16218d4c6773c474011149661470af6e8c90892.zip |
refactor: object_to_vim() cannot fail
Since the parent commit, object_to_vim() can't fail, so callers don't
need to check its result.
Diffstat (limited to 'src/nvim/api/vimscript.c')
-rw-r--r-- | src/nvim/api/vimscript.c | 9 |
1 files changed, 2 insertions, 7 deletions
diff --git a/src/nvim/api/vimscript.c b/src/nvim/api/vimscript.c index 25a34f769c..3bca988030 100644 --- a/src/nvim/api/vimscript.c +++ b/src/nvim/api/vimscript.c @@ -207,9 +207,7 @@ static Object _call_function(String fn, Array args, dict_T *self, Error *err) typval_T vim_args[MAX_FUNC_ARGS + 1]; size_t i = 0; // also used for freeing the variables for (; i < args.size; i++) { - if (!object_to_vim(args.items[i], &vim_args[i], err)) { - goto free_vim_args; - } + object_to_vim(args.items[i], &vim_args[i], err); } // Initialize `force_abort` and `suppress_errthrow` at the top level. @@ -243,7 +241,6 @@ static Object _call_function(String fn, Array args, dict_T *self, Error *err) tv_clear(&rettv); recursive--; -free_vim_args: while (i > 0) { tv_clear(&vim_args[--i]); } @@ -297,9 +294,7 @@ Object nvim_call_dict_function(Object dict, String fn, Array args, Error *err) mustfree = true; break; case kObjectTypeDictionary: - if (!object_to_vim(dict, &rettv, err)) { - goto end; - } + object_to_vim(dict, &rettv, err); break; default: api_set_error(err, kErrorTypeValidation, |