0
0
mirror of https://github.com/vim/vim.git synced 2025-09-23 03:43:49 -04:00

patch 9.0.1470: deferred functions invoked in unexpected order

Problem:    Deferred functions invoked in unexpected order when using :qa and
            autocommands.
Solution:   Call deferred functions for the current funccal before using the
            stack. (closes #12278)
This commit is contained in:
zeertzjq
2023-04-19 14:21:24 +01:00
committed by Bram Moolenaar
parent 960cf9119e
commit 1be4b81bfb
3 changed files with 33 additions and 15 deletions

View File

@@ -704,29 +704,45 @@ endfunc
func Test_defer_quitall_autocmd() func Test_defer_quitall_autocmd()
let lines =<< trim END let lines =<< trim END
autocmd User DeferAutocmdThree qa! func DeferLevelFive()
defer writefile(['5'], 'XQuitallAutocmd', 'a')
func DeferLevelTwo() qa!
call writefile(['text'], 'XQuitallAutocmdTwo', 'D')
doautocmd User DeferAutocmdThree
endfunc endfunc
autocmd User DeferAutocmdTwo ++nested call DeferLevelTwo() autocmd User DeferAutocmdFive call DeferLevelFive()
def DeferLevelOne() def DeferLevelFour()
call writefile(['text'], 'XQuitallAutocmdOne', 'D') defer writefile(['4'], 'XQuitallAutocmd', 'a')
doautocmd User DeferAutocmdTwo doautocmd User DeferAutocmdFive
enddef enddef
func DeferLevelThree()
defer writefile(['3'], 'XQuitallAutocmd', 'a')
call DeferLevelFour()
endfunc
autocmd User DeferAutocmdThree ++nested call DeferLevelThree()
def DeferLevelTwo()
defer writefile(['2'], 'XQuitallAutocmd', 'a')
doautocmd User DeferAutocmdThree
enddef
func DeferLevelOne()
defer writefile(['1'], 'XQuitallAutocmd', 'a')
call DeferLevelTwo()
endfunc
autocmd User DeferAutocmdOne ++nested call DeferLevelOne() autocmd User DeferAutocmdOne ++nested call DeferLevelOne()
doautocmd User DeferAutocmdOne doautocmd User DeferAutocmdOne
END END
call writefile(lines, 'XdeferQuitallAutocmd', 'D') call writefile(lines, 'XdeferQuitallAutocmd', 'D')
let res = system(GetVimCommand() .. ' -X -S XdeferQuitallAutocmd') call system(GetVimCommand() .. ' -X -S XdeferQuitallAutocmd')
call assert_equal(0, v:shell_error) call assert_equal(0, v:shell_error)
call assert_false(filereadable('XQuitallAutocmdOne')) call assert_equal(['5', '4', '3', '2', '1'], readfile('XQuitallAutocmd'))
call assert_false(filereadable('XQuitallAutocmdTwo'))
call delete('XQuitallAutocmd')
endfunc endfunc
func Test_defer_quitall_in_expr_func() func Test_defer_quitall_in_expr_func()

View File

@@ -6144,12 +6144,12 @@ invoke_funccall_defer(funccall_T *fc)
void void
invoke_all_defer(void) invoke_all_defer(void)
{ {
for (funccall_T *fc = current_funccal; fc != NULL; fc = fc->fc_caller)
invoke_funccall_defer(fc);
for (funccal_entry_T *fce = funccal_stack; fce != NULL; fce = fce->next) for (funccal_entry_T *fce = funccal_stack; fce != NULL; fce = fce->next)
for (funccall_T *fc = fce->top_funccal; fc != NULL; fc = fc->fc_caller) for (funccall_T *fc = fce->top_funccal; fc != NULL; fc = fc->fc_caller)
invoke_funccall_defer(fc); invoke_funccall_defer(fc);
for (funccall_T *fc = current_funccal; fc != NULL; fc = fc->fc_caller)
invoke_funccall_defer(fc);
} }
/* /*

View File

@@ -695,6 +695,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 */
/**/
1470,
/**/ /**/
1469, 1469,
/**/ /**/