# HG changeset patch # User Christian Brabandt # Date 1693940403 -7200 # Node ID 256febd1cbf02828fba0871da020279e262e4829 # Parent efd072489401b34499d9b3be6831ab016f926ec5 patch 9.0.1876: Vim9: parsing commands with newlines wrong Commit: https://github.com/vim/vim/commit/00cb247465856eaa546c520b65cf0ccc753ee1cd Author: Christian Brabandt Date: Tue Sep 5 20:46:25 2023 +0200 patch 9.0.1876: Vim9: parsing commands with newlines wrong Problem: Vim9: parsing commands with newlines wrong Solution: Accept a '\n' for parsing lists and command arguments closes: #13015 closes: #13020 Signed-off-by: Christian Brabandt diff --git a/src/ex_docmd.c b/src/ex_docmd.c --- a/src/ex_docmd.c +++ b/src/ex_docmd.c @@ -3954,7 +3954,7 @@ find_ex_command( #ifdef FEAT_EVAL if (eap->cmdidx < CMD_SIZE && vim9 - && !IS_WHITE_OR_NUL(*p) && *p != '\n' && *p != '!' && *p != '|' + && !IS_WHITE_NL_OR_NUL(*p) && *p != '!' && *p != '|' && (eap->cmdidx < 0 || (cmdnames[eap->cmdidx].cmd_argt & EX_NONWHITE_OK) == 0)) { diff --git a/src/list.c b/src/list.c --- a/src/list.c +++ b/src/list.c @@ -1592,7 +1592,7 @@ eval_list(char_u **arg, typval_T *rettv, had_comma = **arg == ','; if (had_comma) { - if (vim9script && !IS_WHITE_OR_NUL((*arg)[1]) && (*arg)[1] != ']') + if (vim9script && !IS_WHITE_NL_OR_NUL((*arg)[1]) && (*arg)[1] != ']') { semsg(_(e_white_space_required_after_str_str), ",", *arg); goto failret; diff --git a/src/macros.h b/src/macros.h --- a/src/macros.h +++ b/src/macros.h @@ -43,6 +43,7 @@ */ #define VIM_ISWHITE(x) ((x) == ' ' || (x) == '\t') #define IS_WHITE_OR_NUL(x) ((x) == ' ' || (x) == '\t' || (x) == NUL) +#define IS_WHITE_NL_OR_NUL(x) ((x) == ' ' || (x) == '\t' || (x) == '\n' || (x) == NUL) /* * LINEEMPTY() - return TRUE if the line is empty diff --git a/src/testdir/test_crash.vim b/src/testdir/test_crash.vim --- a/src/testdir/test_crash.vim +++ b/src/testdir/test_crash.vim @@ -62,7 +62,6 @@ func Test_crash1() let file = 'crash/vim_regsub_both_poc' let args = printf(cmn_args, vim, file) - " using || because this poc causes vim to exit with exitstatus != 0 call term_sendkeys(buf, args .. \ ' && echo "crash 7: [OK]" >> X_crash1_result.txt' .. "\") call TermWait(buf, 1000) diff --git a/src/testdir/test_usercommands.vim b/src/testdir/test_usercommands.vim --- a/src/testdir/test_usercommands.vim +++ b/src/testdir/test_usercommands.vim @@ -772,6 +772,33 @@ func Test_usercmd_with_block() END call v9.CheckScriptFailure(lines, 'E1128:') delcommand BadCommand + + let lines =<< trim END + vim9script + command Cmd { + g:result = [1, + 2] + } + Cmd + END + call v9.CheckScriptSuccess(lines) + call assert_equal([1, 2], g:result) + delcommand Cmd + unlet! g:result + + let lines =<< trim END + vim9script + command Cmd { + g:result = and(0x80, + 0x80) + } + Cmd + END + call v9.CheckScriptSuccess(lines) + call assert_equal(128, g:result) + delcommand Cmd + unlet! g:result + endfunc func Test_delcommand_buffer() diff --git a/src/userfunc.c b/src/userfunc.c --- a/src/userfunc.c +++ b/src/userfunc.c @@ -1879,7 +1879,7 @@ get_func_arguments( argp = skipwhite(argp); if (*argp != ',') break; - if (vim9script && !IS_WHITE_OR_NUL(argp[1])) + if (vim9script && !IS_WHITE_NL_OR_NUL(argp[1])) { if (evaluate) semsg(_(e_white_space_required_after_str_str), ",", argp); diff --git a/src/version.c b/src/version.c --- a/src/version.c +++ b/src/version.c @@ -700,6 +700,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ /**/ + 1876, +/**/ 1875, /**/ 1874,