From 83494b4ac61898f687d6ef9dce4bad5802fb8e51 Mon Sep 17 00:00:00 2001 From: Yegappan Lakshmanan Date: Tue, 20 Jul 2021 17:51:51 +0200 Subject: patch 8.2.3188: Vim9: argument types are not checked at compile time Problem: Vim9: argument types are not checked at compile time. Solution: Add several more type checks, also at runtime. (Yegappan Lakshmanan, closes #8587) --- src/search.c | 26 ++++++++++++++++++-------- 1 file changed, 18 insertions(+), 8 deletions(-) (limited to 'src/search.c') diff --git a/src/search.c b/src/search.c index bbf67d5ce..483265e29 100644 --- a/src/search.c +++ b/src/search.c @@ -1217,7 +1217,8 @@ first_submatch(regmmatch_T *rp) do_search( oparg_T *oap, // can be NULL int dirc, // '/' or '?' - int search_delim, // the delimiter for the search, e.g. '%' in s%regex%replacement% + int search_delim, // the delimiter for the search, e.g. '%' in + // s%regex%replacement% char_u *pat, long count, int options, @@ -1476,11 +1477,11 @@ do_search( msgbuf = trunc; } - #ifdef FEAT_RIGHTLEFT - // The search pattern could be shown on the right in rightleft - // mode, but the 'ruler' and 'showcmd' area use it too, thus - // it would be blanked out again very soon. Show it on the - // left, but do reverse the text. +#ifdef FEAT_RIGHTLEFT + // The search pattern could be shown on the right in + // rightleft mode, but the 'ruler' and 'showcmd' area use + // it too, thus it would be blanked out again very soon. + // Show it on the left, but do reverse the text. if (curwin->w_p_rl && *curwin->w_p_rlc == 's') { char_u *r; @@ -1503,7 +1504,7 @@ do_search( vim_memset(msgbuf + pat_len, ' ', r - msgbuf); } } - #endif +#endif msg_outtrans(msgbuf); msg_clr_eos(); msg_check(); @@ -1548,6 +1549,9 @@ do_search( } } + /* + * The actual search. + */ c = searchit(curwin, curbuf, &pos, NULL, dirc == '/' ? FORWARD : BACKWARD, searchstr, count, spats[0].off.end + (options & @@ -1557,7 +1561,7 @@ do_search( RE_LAST, sia); if (dircp != NULL) - *dircp = search_delim; // restore second '/' or '?' for normal_cmd() + *dircp = search_delim; // restore second '/' or '?' for normal_cmd() if (!shortmess(SHM_SEARCH) && ((dirc == '/' && LT_POS(pos, curwin->w_cursor)) @@ -4794,6 +4798,12 @@ do_fuzzymatch(typval_T *argvars, typval_T *rettv, int retmatchpos) int ret; int matchseq = FALSE; + if (in_vim9script() + && (check_for_list_arg(argvars, 0) == FAIL + || check_for_string_arg(argvars, 1) == FAIL + || check_for_opt_dict_arg(argvars, 2) == FAIL)) + return; + CLEAR_POINTER(&cb); // validate and get the arguments -- cgit v1.2.1