patch 8.2.5089: some functions return a different value on failure
Problem: Some functions return a different value on failure.
Solution: Initialize the return value earlier. (Yegappan Lakshmanan,
closes #10568)
diff --git a/src/evalfunc.c b/src/evalfunc.c
index ff30756..d85e854 100644
--- a/src/evalfunc.c
+++ b/src/evalfunc.c
@@ -8133,6 +8133,9 @@
varnumber_T stride = 1;
int error = FALSE;
+ if (rettv_list_alloc(rettv) != OK)
+ return;
+
if (in_vim9script()
&& (check_for_number_arg(argvars, 0) == FAIL
|| check_for_opt_number_arg(argvars, 1) == FAIL
@@ -8159,7 +8162,7 @@
emsg(_(e_stride_is_zero));
else if (stride > 0 ? end + 1 < start : end - 1 > start)
emsg(_(e_start_past_end));
- else if (rettv_list_alloc(rettv) == OK)
+ else
{
list_T *list = rettv->vval.v_list;