mirror of
https://github.com/vim/vim.git
synced 2025-07-26 11:04:33 -04:00
patch 7.4.1711
Problem: When using try/catch in 'statusline' it is still considered an error and the status line will be disabled. Solution: Check did_emsg instead of called_emsg. (haya14busa, closes #729)
This commit is contained in:
parent
17fe5e1aec
commit
a742e084b6
@ -6779,7 +6779,7 @@ win_redr_status(win_T *wp)
|
|||||||
redraw_custom_statusline(win_T *wp)
|
redraw_custom_statusline(win_T *wp)
|
||||||
{
|
{
|
||||||
static int entered = FALSE;
|
static int entered = FALSE;
|
||||||
int save_called_emsg = called_emsg;
|
int saved_did_emsg = did_emsg;
|
||||||
|
|
||||||
/* When called recursively return. This can happen when the statusline
|
/* When called recursively return. This can happen when the statusline
|
||||||
* contains an expression that triggers a redraw. */
|
* contains an expression that triggers a redraw. */
|
||||||
@ -6787,9 +6787,9 @@ redraw_custom_statusline(win_T *wp)
|
|||||||
return;
|
return;
|
||||||
entered = TRUE;
|
entered = TRUE;
|
||||||
|
|
||||||
called_emsg = FALSE;
|
did_emsg = FALSE;
|
||||||
win_redr_custom(wp, FALSE);
|
win_redr_custom(wp, FALSE);
|
||||||
if (called_emsg)
|
if (did_emsg)
|
||||||
{
|
{
|
||||||
/* When there is an error disable the statusline, otherwise the
|
/* When there is an error disable the statusline, otherwise the
|
||||||
* display is messed up with errors and a redraw triggers the problem
|
* display is messed up with errors and a redraw triggers the problem
|
||||||
@ -6798,7 +6798,7 @@ redraw_custom_statusline(win_T *wp)
|
|||||||
(char_u *)"", OPT_FREE | (*wp->w_p_stl != NUL
|
(char_u *)"", OPT_FREE | (*wp->w_p_stl != NUL
|
||||||
? OPT_LOCAL : OPT_GLOBAL), SID_ERROR);
|
? OPT_LOCAL : OPT_GLOBAL), SID_ERROR);
|
||||||
}
|
}
|
||||||
called_emsg |= save_called_emsg;
|
did_emsg |= saved_did_emsg;
|
||||||
entered = FALSE;
|
entered = FALSE;
|
||||||
}
|
}
|
||||||
#endif
|
#endif
|
||||||
|
@ -22,6 +22,7 @@ source test_reltime.vim
|
|||||||
source test_searchpos.vim
|
source test_searchpos.vim
|
||||||
source test_set.vim
|
source test_set.vim
|
||||||
source test_sort.vim
|
source test_sort.vim
|
||||||
|
source test_statusline.vim
|
||||||
source test_syn_attr.vim
|
source test_syn_attr.vim
|
||||||
source test_timers.vim
|
source test_timers.vim
|
||||||
source test_undolevels.vim
|
source test_undolevels.vim
|
||||||
|
39
src/testdir/test_statusline.vim
Normal file
39
src/testdir/test_statusline.vim
Normal file
@ -0,0 +1,39 @@
|
|||||||
|
function! StatuslineWithCaughtError()
|
||||||
|
let s:func_in_statusline_called = 1
|
||||||
|
try
|
||||||
|
call eval('unknown expression')
|
||||||
|
catch
|
||||||
|
endtry
|
||||||
|
return ''
|
||||||
|
endfunction
|
||||||
|
|
||||||
|
function! StatuslineWithError()
|
||||||
|
let s:func_in_statusline_called = 1
|
||||||
|
call eval('unknown expression')
|
||||||
|
return ''
|
||||||
|
endfunction
|
||||||
|
|
||||||
|
function! Test_caught_error_in_statusline()
|
||||||
|
let s:func_in_statusline_called = 0
|
||||||
|
set laststatus=2
|
||||||
|
let statusline = '%{StatuslineWithCaughtError()}'
|
||||||
|
let &statusline = statusline
|
||||||
|
redrawstatus
|
||||||
|
call assert_true(s:func_in_statusline_called)
|
||||||
|
call assert_equal(statusline, &statusline)
|
||||||
|
set statusline=
|
||||||
|
endfunction
|
||||||
|
|
||||||
|
function! Test_statusline_will_be_disabled_with_error()
|
||||||
|
let s:func_in_statusline_called = 0
|
||||||
|
set laststatus=2
|
||||||
|
let statusline = '%{StatuslineWithError()}'
|
||||||
|
try
|
||||||
|
let &statusline = statusline
|
||||||
|
redrawstatus
|
||||||
|
catch
|
||||||
|
endtry
|
||||||
|
call assert_true(s:func_in_statusline_called)
|
||||||
|
call assert_equal('', &statusline)
|
||||||
|
set statusline=
|
||||||
|
endfunction
|
@ -748,6 +748,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 */
|
||||||
|
/**/
|
||||||
|
1711,
|
||||||
/**/
|
/**/
|
||||||
1710,
|
1710,
|
||||||
/**/
|
/**/
|
||||||
|
Loading…
x
Reference in New Issue
Block a user