mirror of
https://github.com/vim/vim.git
synced 2025-09-25 03:54:15 -04:00
patch 8.2.2725: Vim9: message about compiling is wrong when using try/catch
Problem: Vim9: message about compiling is wrong when using try/catch. Solution: Store the compiling flag with the message. (closes #8071)
This commit is contained in:
@@ -1266,7 +1266,7 @@ do_cmdline(
|
|||||||
if (did_throw)
|
if (did_throw)
|
||||||
{
|
{
|
||||||
char *p = NULL;
|
char *p = NULL;
|
||||||
msglist_T *messages = NULL, *next;
|
msglist_T *messages = NULL;
|
||||||
|
|
||||||
/*
|
/*
|
||||||
* If the uncaught exception is a user exception, report it as an
|
* If the uncaught exception is a user exception, report it as an
|
||||||
@@ -1303,12 +1303,16 @@ do_cmdline(
|
|||||||
{
|
{
|
||||||
do
|
do
|
||||||
{
|
{
|
||||||
next = messages->next;
|
msglist_T *next = messages->next;
|
||||||
|
int save_compiling = estack_compiling;
|
||||||
|
|
||||||
|
estack_compiling = messages->msg_compiling;
|
||||||
emsg(messages->msg);
|
emsg(messages->msg);
|
||||||
vim_free(messages->msg);
|
vim_free(messages->msg);
|
||||||
vim_free(messages->sfile);
|
vim_free(messages->sfile);
|
||||||
vim_free(messages);
|
vim_free(messages);
|
||||||
messages = next;
|
messages = next;
|
||||||
|
estack_compiling = save_compiling;
|
||||||
}
|
}
|
||||||
while (messages != NULL);
|
while (messages != NULL);
|
||||||
}
|
}
|
||||||
|
@@ -292,6 +292,7 @@ cause_errthrow(
|
|||||||
// reaching do_errthrow().
|
// reaching do_errthrow().
|
||||||
elem->sfile = estack_sfile(ESTACK_NONE);
|
elem->sfile = estack_sfile(ESTACK_NONE);
|
||||||
elem->slnum = SOURCING_LNUM;
|
elem->slnum = SOURCING_LNUM;
|
||||||
|
elem->msg_compiling = estack_compiling;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
@@ -971,11 +971,12 @@ typedef struct {
|
|||||||
typedef struct msglist msglist_T;
|
typedef struct msglist msglist_T;
|
||||||
struct msglist
|
struct msglist
|
||||||
{
|
{
|
||||||
|
msglist_T *next; // next of several messages in a row
|
||||||
char *msg; // original message, allocated
|
char *msg; // original message, allocated
|
||||||
char *throw_msg; // msg to throw: usually original one
|
char *throw_msg; // msg to throw: usually original one
|
||||||
char_u *sfile; // value from estack_sfile(), allocated
|
char_u *sfile; // value from estack_sfile(), allocated
|
||||||
long slnum; // line number for "sfile"
|
long slnum; // line number for "sfile"
|
||||||
msglist_T *next; // next of several messages in a row
|
int msg_compiling; // saved value of estack_compiling
|
||||||
};
|
};
|
||||||
|
|
||||||
/*
|
/*
|
||||||
|
@@ -18,6 +18,7 @@ func Test_compiling_error()
|
|||||||
CheckRunVimInTerminal
|
CheckRunVimInTerminal
|
||||||
|
|
||||||
call TestCompilingError()
|
call TestCompilingError()
|
||||||
|
call TestCompilingErrorInTry()
|
||||||
endfunc
|
endfunc
|
||||||
|
|
||||||
def TestCompilingError()
|
def TestCompilingError()
|
||||||
@@ -28,15 +29,49 @@ def TestCompilingError()
|
|||||||
enddef
|
enddef
|
||||||
defcompile
|
defcompile
|
||||||
END
|
END
|
||||||
call writefile(lines, 'XTest_compile_error')
|
writefile(lines, 'XTest_compile_error')
|
||||||
var buf = RunVimInTerminal('-S XTest_compile_error',
|
var buf = RunVimInTerminal('-S XTest_compile_error',
|
||||||
{rows: 10, wait_for_ruler: 0})
|
{rows: 10, wait_for_ruler: 0})
|
||||||
call WaitForAssert(() => assert_match('Error detected while compiling command line.*Fails.*Variable not found: nothing',
|
WaitForAssert(() => assert_match('Error detected while compiling command line.*Fails.*Variable not found: nothing',
|
||||||
Term_getlines(buf, range(1, 9))))
|
Term_getlines(buf, range(1, 9))))
|
||||||
|
|
||||||
# clean up
|
# clean up
|
||||||
call StopVimInTerminal(buf)
|
StopVimInTerminal(buf)
|
||||||
call delete('XTest_compile_error')
|
delete('XTest_compile_error')
|
||||||
|
enddef
|
||||||
|
|
||||||
|
def TestCompilingErrorInTry()
|
||||||
|
var dir = 'Xdir/autoload'
|
||||||
|
mkdir(dir, 'p')
|
||||||
|
|
||||||
|
var lines =<< trim END
|
||||||
|
vim9script
|
||||||
|
def script#OnlyCompiled()
|
||||||
|
g:runtime = 'yes'
|
||||||
|
invalid
|
||||||
|
enddef
|
||||||
|
END
|
||||||
|
writefile(lines, dir .. '/script.vim')
|
||||||
|
|
||||||
|
lines =<< trim END
|
||||||
|
vim9script
|
||||||
|
todo
|
||||||
|
try
|
||||||
|
script#OnlyCompiled()
|
||||||
|
catch /nothing/
|
||||||
|
endtry
|
||||||
|
END
|
||||||
|
lines[1] = 'set rtp=' .. getcwd() .. '/Xdir'
|
||||||
|
writefile(lines, 'XTest_compile_error')
|
||||||
|
|
||||||
|
var buf = RunVimInTerminal('-S XTest_compile_error', {rows: 10, wait_for_ruler: 0})
|
||||||
|
WaitForAssert(() => assert_match('Error detected while compiling command line.*function script#OnlyCompiled.*Invalid command: invalid',
|
||||||
|
Term_getlines(buf, range(1, 9))))
|
||||||
|
|
||||||
|
# clean up
|
||||||
|
StopVimInTerminal(buf)
|
||||||
|
delete('XTest_compile_error')
|
||||||
|
delete('Xdir', 'rf')
|
||||||
enddef
|
enddef
|
||||||
|
|
||||||
def CallRecursive(n: number): number
|
def CallRecursive(n: number): number
|
||||||
|
@@ -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 */
|
||||||
|
/**/
|
||||||
|
2725,
|
||||||
/**/
|
/**/
|
||||||
2724,
|
2724,
|
||||||
/**/
|
/**/
|
||||||
|
Reference in New Issue
Block a user