# HG changeset patch # User Bram Moolenaar # Date 1615061702 -3600 # Node ID 780dec2ffa6b1361e22a31dd74fb99e1154174e4 # Parent 425c76b8e17fb8046368094516c7001fa1159afa patch 8.2.2575: Vim9: a function name with "->" in the next line doesn't work Commit: https://github.com/vim/vim/commit/6914e87d3c0387fdcbb117a39e1f6d1fac0ee2e3 Author: Bram Moolenaar Date: Sat Mar 6 21:01:09 2021 +0100 patch 8.2.2575: Vim9: a function name with "->" in the next line doesn't work Problem: Vim9: a function name with "->" in the next line doesn't work. Solution: Recognize a function name by itself. (closes https://github.com/vim/vim/issues/7770) diff --git a/src/testdir/test_vim9_cmd.vim b/src/testdir/test_vim9_cmd.vim --- a/src/testdir/test_vim9_cmd.vim +++ b/src/testdir/test_vim9_cmd.vim @@ -357,6 +357,25 @@ def Test_method_call_linebreak() lines =<< trim END new + def Foo(): string + return 'the text' + enddef + def Bar(F: func): string + return F() + enddef + def Test() + Foo + ->Bar() + ->setline(1) + enddef + Test() + assert_equal('the text', getline(1)) + bwipe! + END + CheckDefAndScriptSuccess(lines) + + lines =<< trim END + new g:shortlist ->copy() ->setline(1) diff --git a/src/version.c b/src/version.c --- a/src/version.c +++ b/src/version.c @@ -751,6 +751,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ /**/ + 2575, +/**/ 2574, /**/ 2573, diff --git a/src/vim9compile.c b/src/vim9compile.c --- a/src/vim9compile.c +++ b/src/vim9compile.c @@ -387,6 +387,26 @@ variable_exists(char_u *name, size_t len } /* + * Return TRUE if "name" is a local variable, argument, script variable, + * imported or function. + */ + static int +item_exists(char_u *name, size_t len, cctx_T *cctx) +{ + int is_global; + + if (variable_exists(name, len, cctx)) + return TRUE; + + // Find a function, so that a following "->" works. Skip "g:" before a + // function name. + // Do not check for an internal function, since it might also be a + // valid command, such as ":split" versuse "split()". + is_global = (name[0] == 'g' && name[1] == ':'); + return find_func(is_global ? name + 2 : name, is_global, cctx) != NULL; +} + +/* * Check if "p[len]" is already defined, either in script "import_sid" or in * compilation context "cctx". "cctx" is NULL at the script level. * Does not check the global namespace. @@ -728,7 +748,7 @@ get_compare_isn(exprtype_T exprtype, var } else if (type1 == VAR_ANY || type2 == VAR_ANY || ((type1 == VAR_NUMBER || type1 == VAR_FLOAT) - && (type2 == VAR_NUMBER || type2 ==VAR_FLOAT))) + && (type2 == VAR_NUMBER || type2 == VAR_FLOAT))) isntype = ISN_COMPAREANY; if ((exprtype == EXPR_IS || exprtype == EXPR_ISNOT) @@ -8399,8 +8419,7 @@ compile_def_function( } } p = find_ex_command(&ea, NULL, starts_with_colon ? NULL - : (int (*)(char_u *, size_t, cctx_T *))variable_exists, - &cctx); + : (int (*)(char_u *, size_t, cctx_T *))item_exists, &cctx); if (p == NULL) {