mirror of
https://github.com/vim/vim.git
synced 2025-09-26 04:04:07 -04:00
patch 8.2.3404: Vim9: no error for white space before "("
Problem: Vim9: no error for white space before "(". Solution: Give an error, like in a compiled function.
This commit is contained in:
@@ -467,6 +467,10 @@ def Test_call_varargs()
|
|||||||
MyVarargs('one', 'two', 'three')->assert_equal('one,two,three')
|
MyVarargs('one', 'two', 'three')->assert_equal('one,two,three')
|
||||||
enddef
|
enddef
|
||||||
|
|
||||||
|
def Test_call_white_space()
|
||||||
|
CheckDefAndScriptFailure2(["call Test ('text')"], 'E476:', 'E1068:')
|
||||||
|
enddef
|
||||||
|
|
||||||
def MyDefaultArgs(name = 'string'): string
|
def MyDefaultArgs(name = 'string'): string
|
||||||
return name
|
return name
|
||||||
enddef
|
enddef
|
||||||
|
@@ -4921,13 +4921,16 @@ ex_call(exarg_T *eap)
|
|||||||
// Skip white space to allow ":call func ()". Not good, but required for
|
// Skip white space to allow ":call func ()". Not good, but required for
|
||||||
// backward compatibility.
|
// backward compatibility.
|
||||||
startarg = skipwhite(arg);
|
startarg = skipwhite(arg);
|
||||||
rettv.v_type = VAR_UNKNOWN; // clear_tv() uses this
|
|
||||||
|
|
||||||
if (*startarg != '(')
|
if (*startarg != '(')
|
||||||
{
|
{
|
||||||
semsg(_(e_missing_paren), eap->arg);
|
semsg(_(e_missing_paren), eap->arg);
|
||||||
goto end;
|
goto end;
|
||||||
}
|
}
|
||||||
|
if (in_vim9script() && startarg > arg)
|
||||||
|
{
|
||||||
|
semsg(_(e_no_white_space_allowed_before_str_str), "(", eap->arg);
|
||||||
|
goto end;
|
||||||
|
}
|
||||||
|
|
||||||
/*
|
/*
|
||||||
* When skipping, evaluate the function once, to find the end of the
|
* When skipping, evaluate the function once, to find the end of the
|
||||||
@@ -4969,6 +4972,7 @@ ex_call(exarg_T *eap)
|
|||||||
funcexe.partial = partial;
|
funcexe.partial = partial;
|
||||||
funcexe.selfdict = fudi.fd_dict;
|
funcexe.selfdict = fudi.fd_dict;
|
||||||
funcexe.check_type = type;
|
funcexe.check_type = type;
|
||||||
|
rettv.v_type = VAR_UNKNOWN; // clear_tv() uses this
|
||||||
if (get_func_tv(name, -1, &rettv, &arg, &evalarg, &funcexe) == FAIL)
|
if (get_func_tv(name, -1, &rettv, &arg, &evalarg, &funcexe) == FAIL)
|
||||||
{
|
{
|
||||||
failed = TRUE;
|
failed = TRUE;
|
||||||
|
@@ -755,6 +755,8 @@ static char *(features[]) =
|
|||||||
|
|
||||||
static int included_patches[] =
|
static int included_patches[] =
|
||||||
{ /* Add new patch number below this line */
|
{ /* Add new patch number below this line */
|
||||||
|
/**/
|
||||||
|
3404,
|
||||||
/**/
|
/**/
|
||||||
3403,
|
3403,
|
||||||
/**/
|
/**/
|
||||||
|
Reference in New Issue
Block a user