adam/vim
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

patch 9.1.1211: TabClosedPre is triggered just before the tab is being freed

Browse Source 
Problem:  TabClosedPre is triggered just before the tab is being freed,
          which limited its functionality.
Solution: Trigger it a bit earlier and also on :tabclose and :tabonly
          (Jim Zhou)

closes: #16890

Signed-off-by: Jim Zhou <jimzhouzzy@gmail.com>
Signed-off-by: Christian Brabandt <cb@256bit.org>
This commit is contained in:
Jim Zhou
2025-03-16 20:24:57 +01:00
committed by Christian Brabandt
parent 21ac3a49b5
commit bcf66e0141
5 changed files with 53 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

22
src/window.c
View File

@ -2978,10 +2978,15 @@ trigger_winclosed(win_T *win)
recursive = FALSE;
}
static void
trigger_tabclosedpre(tabpage_T *tp)
/*
* directly is TRUE if the window is closed by ':tabclose' or ':tabonly'.
* This allows saving the session before closing multi-window tab.
*/
void
trigger_tabclosedpre(tabpage_T *tp, int directly)
{
static int recursive = FALSE;
static int skip = FALSE;
tabpage_T *ptp = curtab;
// Quickly return when no TabClosedPre autocommands to be executed or
@ -2989,8 +2994,19 @@ trigger_tabclosedpre(tabpage_T *tp)
if (!has_tabclosedpre() || recursive)
return;
// Skip if the event have been triggered by ':tabclose' recently
if (skip)
{
skip = FALSE;
return;
}
if (valid_tabpage(tp))
{
goto_tabpage_tp(tp, FALSE, FALSE);
if (directly)
skip = TRUE;
}
recursive = TRUE;
window_layout_lock();
apply_autocmds(EVENT_TABCLOSEDPRE, NULL, NULL, FALSE, NULL);
@ -3382,7 +3398,7 @@ win_close_othertab(win_T *win, int free_buf, tabpage_T *tp)
if (tp->tp_firstwin == tp->tp_lastwin)
{
trigger_tabclosedpre(tp);
trigger_tabclosedpre(tp, FALSE);
// autocmd may have freed the window already.
if (!win_valid_any_tab(win))
return;