From 5582ef14384525e8cec86016876d97a6b32dd548 Mon Sep 17 00:00:00 2001 From: Bram Moolenaar Date: Wed, 14 Sep 2016 22:16:13 +0200 Subject: [PATCH] patch 8.0.0004 Problem: A string argument for function() that is not a function name results in an error message with NULL. (Christian Brabandt) Solution: Use the argument for the error message. --- src/evalfunc.c | 2 +- src/testdir/test_expr.vim | 3 +++ src/version.c | 2 ++ 3 files changed, 6 insertions(+), 1 deletion(-) diff --git a/src/evalfunc.c b/src/evalfunc.c index 7dd5c2a4d..3207441c2 100644 --- a/src/evalfunc.c +++ b/src/evalfunc.c @@ -3612,7 +3612,7 @@ common_function(typval_T *argvars, typval_T *rettv, int is_funcref) if (s == NULL || *s == NUL || (use_string && VIM_ISDIGIT(*s)) || (is_funcref && trans_name == NULL)) - EMSG2(_(e_invarg2), s); + EMSG2(_(e_invarg2), use_string ? get_tv_string(&argvars[0]) : s); /* Don't check an autoload name for existence here. */ else if (trans_name != NULL && (is_funcref ? find_func(trans_name) == NULL diff --git a/src/testdir/test_expr.vim b/src/testdir/test_expr.vim index 108ee508e..dff8e36f4 100644 --- a/src/testdir/test_expr.vim +++ b/src/testdir/test_expr.vim @@ -439,6 +439,9 @@ func Test_function_with_funcref() let s:fref = function(s:f) call assert_equal(v:t_string, s:fref('x')) call assert_fails("call function('s:f')", 'E700:') + + call assert_fails("call function('foo()')", 'E475:') + call assert_fails("call function('foo()')", 'foo()') endfunc func Test_funcref() diff --git a/src/version.c b/src/version.c index dcf02c288..563411438 100644 --- a/src/version.c +++ b/src/version.c @@ -764,6 +764,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ +/**/ + 4, /**/ 3, /**/ -- 2.50.1