aniani/vim
0
0
Fork 1
mirror of https://github.com/vim/vim.git synced 2025-09-24 03:44:06 -04:00
Code Activity

patch 8.2.2706: Vim9: wrong line number reported for boolean operator

Browse Source 
Problem:    Vim9: wrong line number reported for boolean operator.
Solution:   Use the line number before skipping over line break.
            (closes #8058)
This commit is contained in:
Bram Moolenaar
2021-04-03 21:47:07 +02:00
parent 77709b194c
commit a7511c0f4f
3 changed files with 33 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

30
src/testdir/test_vim9_expr.vim
View File

@@ -467,16 +467,30 @@ def Test_expr3_vimscript()
CheckScriptFailure(lines, 'E1004:', 2) CheckScriptFailure(lines, 'E1004:', 2)
enddef enddef
func Test_expr3_fails() def Test_expr3_fails()
let msg = "White space required before and after '&&'" var msg = "White space required before and after '&&'"
call CheckDefFailure(["var x = 1&&2"], msg, 1) CheckDefFailure(["var x = 1&&2"], msg, 1)
call CheckDefFailure(["var x = 1 &&2"], msg, 1) CheckDefFailure(["var x = 1 &&2"], msg, 1)
call CheckDefFailure(["var x = 1&& 2"], msg, 1) CheckDefFailure(["var x = 1&& 2"], msg, 1)
call CheckDefFailure(["if 'yes' && 0", 'echo 0', 'endif'], 'E1012: Type mismatch; expected bool but got string', 1) CheckDefFailure(["if 'yes' && 0", 'echo 0', 'endif'], 'E1012: Type mismatch; expected bool but got string', 1)
call CheckDefExecFailure(['assert_equal(false, Record(1) && Record(4) && Record(0))'], 'E1023: Using a Number as a Bool: 4', 1) CheckDefExecFailure(['assert_equal(false, Record(1) && Record(4) && Record(0))'], 'E1023: Using a Number as a Bool: 4', 1)
endfunc
var lines =<< trim END
if 3
&& true
endif
END
CheckDefExecFailure(lines, 'E1023:', 1)
lines =<< trim END
if 'yes'
&& true
endif
END
CheckDefFailure(lines, 'E1012:', 1)
enddef
" global variables to use for tests with the "any" type " global variables to use for tests with the "any" type
let atrue = v:true let atrue = v:true

2
src/version.c
View File

@@ -750,6 +750,8 @@ static char *(features[]) =
static int included_patches[] = static int included_patches[] =
{ /* Add new patch number below this line */ { /* Add new patch number below this line */
/**/
2706,
/**/ /**/
2705, 2705,
/**/ /**/

9
src/vim9compile.c
View File

@@ -4772,6 +4772,10 @@ compile_and_or(
ga_init2(&end_ga, sizeof(int), 10); ga_init2(&end_ga, sizeof(int), 10);
while (p[0] == opchar && p[1] == opchar) while (p[0] == opchar && p[1] == opchar)
{ {
long start_lnum = SOURCING_LNUM;
int start_ctx_lnum = cctx->ctx_lnum;
int save_lnum;
if (next != NULL) if (next != NULL)
{ {
*arg = next_line_from_context(cctx, TRUE); *arg = next_line_from_context(cctx, TRUE);
@@ -4790,11 +4794,16 @@ compile_and_or(
generate_ppconst(cctx, ppconst); generate_ppconst(cctx, ppconst);
// Every part must evaluate to a bool. // Every part must evaluate to a bool.
SOURCING_LNUM = start_lnum;
save_lnum = cctx->ctx_lnum;
cctx->ctx_lnum = start_ctx_lnum;
if (bool_on_stack(cctx) == FAIL) if (bool_on_stack(cctx) == FAIL)
{ {
cctx->ctx_lnum = save_lnum;
ga_clear(&end_ga); ga_clear(&end_ga);
return FAIL; return FAIL;
} }
cctx->ctx_lnum = save_lnum;
if (ga_grow(&end_ga, 1) == FAIL) if (ga_grow(&end_ga, 1) == FAIL)
{ {