mirror of
https://github.com/vim/vim.git
synced 2025-10-24 08:54:47 -04:00
patch 9.1.1393: missing test for switching buffers and reusing curbuf
Problem: The check in buf_freeall that restores curwin subtly prevents
leaving an unloaded buffer in a window when reusing curbuf, if
autocommands switch to a different buffer.
Solution: Add a test case that covers this. Also ensure splitting isn't
possible, as that could do the same (Sean Dewar)
closes: #17325
Signed-off-by: Sean Dewar <6256228+seandewar@users.noreply.github.com>
Signed-off-by: Christian Brabandt <cb@256bit.org>
This commit is contained in:
committed by
Christian Brabandt
parent
c49cb93a86
commit
31be82e66d
@@ -5415,4 +5415,31 @@ func Test_reuse_curbuf_leak()
|
||||
%bw!
|
||||
endfunc
|
||||
|
||||
func Test_reuse_curbuf_switch()
|
||||
edit asdf
|
||||
let s:asdf_win = win_getid()
|
||||
new
|
||||
let other_buf = bufnr()
|
||||
let other_win = win_getid()
|
||||
augroup testing
|
||||
autocmd!
|
||||
autocmd BufUnload * ++once let s:triggered = 1
|
||||
\| call assert_fails('split', 'E1159:')
|
||||
\| call win_gotoid(s:asdf_win)
|
||||
augroup END
|
||||
|
||||
" Check BufUnload changing curbuf does not cause buflist_new to create a new
|
||||
" buffer while leaving "other_buf" unloaded in a window.
|
||||
enew
|
||||
call assert_equal(1, s:triggered)
|
||||
call assert_equal(other_buf, bufnr())
|
||||
call assert_equal(other_win, win_getid())
|
||||
call assert_equal(1, win_findbuf(other_buf)->len())
|
||||
call assert_equal(1, bufloaded(other_buf))
|
||||
|
||||
unlet! s:asdf_win s:triggered
|
||||
call CleanUpTestAuGroup()
|
||||
%bw!
|
||||
endfunc
|
||||
|
||||
" vim: shiftwidth=2 sts=2 expandtab
|
||||
|
||||
@@ -709,6 +709,8 @@ static char *(features[]) =
|
||||
|
||||
static int included_patches[] =
|
||||
{ /* Add new patch number below this line */
|
||||
/**/
|
||||
1393,
|
||||
/**/
|
||||
1392,
|
||||
/**/
|
||||
|
||||
Reference in New Issue
Block a user