# HG changeset patch # User Bram Moolenaar # Date 1651750205 -7200 # Node ID 9a7a2908e1a871e1818add769d6f8713dee7d0b7 # Parent 0c6c580c01b437264a15d4e3d31e8d820eb2e3aa patch 8.2.4869: expression in command block does not look after NL Commit: https://github.com/vim/vim/commit/e442d59f6d6732c226d636067c07b7342838b36b Author: Bram Moolenaar Date: Thu May 5 12:20:28 2022 +0100 patch 8.2.4869: expression in command block does not look after NL Problem: Expression in command block does not look after NL. Solution: Skip over NL to check what follows. (closes https://github.com/vim/vim/issues/10358) diff --git a/src/eval.c b/src/eval.c --- a/src/eval.c +++ b/src/eval.c @@ -1803,7 +1803,7 @@ skip_for_lines(void *fi_void, evalarg_T int i; for (i = 0; i < fi->fi_break_count; ++i) - eval_next_line(evalarg); + eval_next_line(NULL, evalarg); } /* @@ -2153,7 +2153,7 @@ getline_peek_skip_comments(evalarg_T *ev p = skipwhite(next); if (*p != NUL && !vim9_comment_start(p)) return next; - if (eval_next_line(evalarg) == NULL) + if (eval_next_line(NULL, evalarg) == NULL) break; } return NULL; @@ -2176,11 +2176,14 @@ eval_next_non_blank(char_u *arg, evalarg if (in_vim9script() && evalarg != NULL && (evalarg->eval_cookie != NULL || evalarg->eval_cctx != NULL) - && (*p == NUL || (vim9_comment_start(p) && VIM_ISWHITE(p[-1])))) + && (*p == NUL || *p == NL + || (vim9_comment_start(p) && VIM_ISWHITE(p[-1])))) { char_u *next; - if (evalarg->eval_cookie != NULL) + if (*p == NL) + next = p + 1; + else if (evalarg->eval_cookie != NULL) next = getline_peek_skip_comments(evalarg); else next = peek_next_line_from_context(evalarg->eval_cctx); @@ -2199,11 +2202,14 @@ eval_next_non_blank(char_u *arg, evalarg * Only called for Vim9 script. */ char_u * -eval_next_line(evalarg_T *evalarg) +eval_next_line(char_u *arg, evalarg_T *evalarg) { garray_T *gap = &evalarg->eval_ga; char_u *line; + if (arg != NULL && *arg == NL) + return skipwhite(arg + 1); + if (evalarg->eval_cookie != NULL) line = evalarg->eval_getline(0, evalarg->eval_cookie, 0, GETLINE_CONCAT_ALL); @@ -2253,7 +2259,7 @@ skipwhite_and_linebreak(char_u *arg, eva return skipwhite(arg); eval_next_non_blank(p, evalarg, &getnext); if (getnext) - return eval_next_line(evalarg); + return eval_next_line(arg, evalarg); return p; } @@ -2449,7 +2455,7 @@ eval1(char_u **arg, typval_T *rettv, eva evaluate = evalarg_used->eval_flags & EVAL_EVALUATE; if (getnext) - *arg = eval_next_line(evalarg_used); + *arg = eval_next_line(*arg, evalarg_used); else { if (evaluate && vim9script && !VIM_ISWHITE(p[-1])) @@ -2515,7 +2521,7 @@ eval1(char_u **arg, typval_T *rettv, eva return FAIL; } if (getnext) - *arg = eval_next_line(evalarg_used); + *arg = eval_next_line(*arg, evalarg_used); else { if (evaluate && vim9script && !VIM_ISWHITE(p[-1])) @@ -2621,7 +2627,7 @@ eval2(char_u **arg, typval_T *rettv, eva while (p[0] == '|' && p[1] == '|') { if (getnext) - *arg = eval_next_line(evalarg_used); + *arg = eval_next_line(*arg, evalarg_used); else { if (evaluate && vim9script && !VIM_ISWHITE(p[-1])) @@ -2747,7 +2753,7 @@ eval3(char_u **arg, typval_T *rettv, eva while (p[0] == '&' && p[1] == '&') { if (getnext) - *arg = eval_next_line(evalarg_used); + *arg = eval_next_line(*arg, evalarg_used); else { if (evaluate && vim9script && !VIM_ISWHITE(p[-1])) @@ -2864,7 +2870,7 @@ eval4(char_u **arg, typval_T *rettv, eva if (getnext) { - *arg = eval_next_line(evalarg); + *arg = eval_next_line(*arg, evalarg); p = *arg; } else if (evaluate && vim9script && !VIM_ISWHITE(**arg)) @@ -3026,7 +3032,7 @@ eval5(char_u **arg, typval_T *rettv, eva evaluate = evalarg == NULL ? 0 : (evalarg->eval_flags & EVAL_EVALUATE); oplen = (concat && p[1] == '.') ? 2 : 1; if (getnext) - *arg = eval_next_line(evalarg); + *arg = eval_next_line(*arg, evalarg); else { if (evaluate && vim9script && !VIM_ISWHITE(**arg)) @@ -3258,7 +3264,7 @@ eval6( evaluate = evalarg == NULL ? 0 : (evalarg->eval_flags & EVAL_EVALUATE); if (getnext) - *arg = eval_next_line(evalarg); + *arg = eval_next_line(*arg, evalarg); else { if (evaluate && in_vim9script() && !VIM_ISWHITE(**arg)) @@ -6150,7 +6156,7 @@ handle_subscript( || ASCII_ISALPHA(in_vim9script() ? *skipwhite(p + 2) : p[2]))))) { - *arg = eval_next_line(evalarg); + *arg = eval_next_line(*arg, evalarg); p = *arg; check_white = FALSE; } diff --git a/src/proto/eval.pro b/src/proto/eval.pro --- a/src/proto/eval.pro +++ b/src/proto/eval.pro @@ -32,7 +32,7 @@ void free_for_info(void *fi_void); void set_context_for_expression(expand_T *xp, char_u *arg, cmdidx_T cmdidx); int pattern_match(char_u *pat, char_u *text, int ic); char_u *eval_next_non_blank(char_u *arg, evalarg_T *evalarg, int *getnext); -char_u *eval_next_line(evalarg_T *evalarg); +char_u *eval_next_line(char_u *arg, evalarg_T *evalarg); char_u *skipwhite_and_linebreak(char_u *arg, evalarg_T *evalarg); void init_evalarg(evalarg_T *evalarg); void clear_evalarg(evalarg_T *evalarg, exarg_T *eap); 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 @@ -695,6 +695,20 @@ func Test_usercmd_with_block() delcommand HelloThere let lines =<< trim END + command EchoCond { + const test: string = true + ? 'true' + : 'false' + g:result = test + } + EchoCond + END + call v9.CheckScriptSuccess(lines) + call assert_equal('true', g:result) + delcommand EchoCond + unlet g:result + + let lines =<< trim END command BadCommand { echo { 'key': 'value', diff --git a/src/version.c b/src/version.c --- a/src/version.c +++ b/src/version.c @@ -747,6 +747,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ /**/ + 4869, +/**/ 4868, /**/ 4867, diff --git a/src/vim9script.c b/src/vim9script.c --- a/src/vim9script.c +++ b/src/vim9script.c @@ -589,7 +589,7 @@ handle_import( char_u *p; if (getnext) - arg = eval_next_line(evalarg); + arg = eval_next_line(expr_end, evalarg); else arg = nextarg;