]> granicus.if.org Git - vim/commitdiff
patch 8.2.2396: Vim9: no white space allowed before "->" v8.2.2396
authorBram Moolenaar <Bram@vim.org>
Sat, 23 Jan 2021 13:22:16 +0000 (14:22 +0100)
committerBram Moolenaar <Bram@vim.org>
Sat, 23 Jan 2021 13:22:16 +0000 (14:22 +0100)
Problem:    Vim9: no white space allowed before "->".
Solution:   Allow for white space. (closes #7725)

src/eval.c
src/ex_docmd.c
src/testdir/test_vim9_cmd.vim
src/version.c

index 3e316bdb1061a5d568b3a38759be54b6d5e853fd..ebd25077c3ede3b3701a608f7c7dba2351640b5c 100644 (file)
@@ -3671,7 +3671,7 @@ call_func_rettv(
 
 /*
  * Evaluate "->method()".
- * "*arg" points to the '-'.
+ * "*arg" points to "method".
  * Returns FAIL or OK. "*arg" is advanced to after the ')'.
  */
     static int
@@ -3686,8 +3686,6 @@ eval_lambda(
     typval_T   base = *rettv;
     int                ret;
 
-    // Skip over the ->.
-    *arg += 2;
     rettv->v_type = VAR_UNKNOWN;
 
     if (**arg == '{')
@@ -3735,7 +3733,7 @@ eval_lambda(
 
 /*
  * Evaluate "->method()".
- * "*arg" points to the '-'.
+ * "*arg" points to "method".
  * Returns FAIL or OK. "*arg" is advanced to after the ')'.
  */
     static int
@@ -3753,8 +3751,6 @@ eval_method(
     int                evaluate = evalarg != NULL
                                      && (evalarg->eval_flags & EVAL_EVALUATE);
 
-    // Skip over the ->.
-    *arg += 2;
     rettv->v_type = VAR_UNKNOWN;
 
     name = *arg;
@@ -5765,10 +5761,10 @@ handle_subscript(
        }
        else if (p[0] == '-' && p[1] == '>')
        {
-           *arg = p;
+           *arg = skipwhite(p + 2);
            if (ret == OK)
            {
-               if (((*arg)[2] == '{' && !in_vim9script()) || (*arg)[2] == '(')
+               if ((**arg == '{' && !in_vim9script()) || **arg == '(')
                    // expr->{lambda}() or expr->(lambda)()
                    ret = eval_lambda(arg, rettv, evalarg, verbose);
                else
index 9562e6be3a8323aeee5d6ebb8d1a7baf66edf2bf..b9ae13d0a8d1c816a0a9f058df77cca1467d8bd3 100644 (file)
@@ -3313,8 +3313,9 @@ find_ex_command(
        if (vim_strchr((char_u *)"{('[\"@", *p) != NULL
               || ((p = to_name_const_end(pskip)) > eap->cmd && *p != NUL))
        {
-           int oplen;
-           int heredoc;
+           int     oplen;
+           int     heredoc;
+           char_u  *swp = skipwhite(p);
 
            if (
                // "(..." is an expression.
@@ -3332,7 +3333,7 @@ find_ex_command(
                         || eap->cmd[1] == ':'
                            )
                            // "varname->func()" is an expression.
-                       : (*p == '-' && p[1] == '>')))
+                       : (*swp == '-' && swp[1] == '>')))
            {
                if (*eap->cmd == '{' && ends_excmd(*skipwhite(eap->cmd + 1)))
                {
index 12b9bf191b0fb4792e0f6a3384489eee7d6613db..735f48807d81bad3aa3db54fe74adc63e3abbee4 100644 (file)
@@ -319,6 +319,20 @@ def Test_method_call_linebreak()
   CheckScriptSuccess(lines)
 enddef
 
+def Test_method_call_whitespace()
+  var lines =<< trim END
+    new
+    var yank = 'text'
+    yank->setline(1)
+    yank  ->setline(2)
+    yank->  setline(3)
+    yank  ->  setline(4)
+    assert_equal(['text', 'text', 'text', 'text'], getline(1, 4))
+    bwipe!
+  END
+  CheckDefAndScriptSuccess(lines)
+enddef
+
 def Test_skipped_expr_linebreak()
   if 0
     var x = []
index 7942e743f2f28f1b0c898f0aa3e64f7daf9d49ad..af1f7d2959c1dcea50dca2892a569412fa9a4a1e 100644 (file)
@@ -750,6 +750,8 @@ static char *(features[]) =
 
 static int included_patches[] =
 {   /* Add new patch number below this line */
+/**/
+    2396,
 /**/
     2395,
 /**/