diff options
| author | Christian Brabandt <cb@256bit.org> | 2023-12-16 13:55:32 +0100 |
|---|---|---|
| committer | Christian Brabandt <cb@256bit.org> | 2023-12-16 13:55:32 +0100 |
| commit | df12e39b8b9dd39056e22b452276622cb7b617fd (patch) | |
| tree | 01462ce0832fb7a78def6d49d2b1ed857c8e77f7 /src/globals.h | |
| parent | 3afc9f2556faf2574d7950d879f2bf61612f3367 (diff) | |
patch 9.0.2168: Moving tabpages on :drop may cause an endless loopv9.0.2168
Problem: Moving tabpages on :drop may cause an endless loop
Solution: Disallow moving tabpages on :drop when cleaning up the arglist
first
Moving tabpages during drop command may cause an endless loop
When executing a :tab drop command, Vim will close all windows not in
the argument list. This triggers various autocommands. If a user has
created an 'au Tabenter * :tabmove -' autocommand, this can cause Vim to
end up in an endless loop, when trying to iterate over all tabs (which
would trigger the tabmove autocommand, which will change the tpnext
pointer, etc).
So instead of blocking all autocommands before we actually try to edit
the given file, lets simply disallow to move tabpages around. Otherwise,
we may change the expected number of events triggered during a :drop
command, which users may rely on (there is actually a test, that expects
various TabLeave/TabEnter autocommands) and would therefore be a
backwards incompatible change.
Don't make this an error, as this could trigger several times during the
drop command, but silently ignore the :tabmove command in this case (and
it should in fact finally trigger successfully when loading the given
file in a new tab). So let's just be quiet here instead.
fixes: #13676
closes: #13686
Signed-off-by: Christian Brabandt <cb@256bit.org>
Diffstat (limited to 'src/globals.h')
| -rw-r--r-- | src/globals.h | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/src/globals.h b/src/globals.h index d54e311837..c03fe97db9 100644 --- a/src/globals.h +++ b/src/globals.h @@ -792,8 +792,9 @@ EXTERN int is_mac_terminal INIT(= FALSE); // recognized Terminal.app #endif EXTERN int autocmd_busy INIT(= FALSE); // Is apply_autocmds() busy? -EXTERN int autocmd_no_enter INIT(= FALSE); // *Enter autocmds disabled -EXTERN int autocmd_no_leave INIT(= FALSE); // *Leave autocmds disabled +EXTERN int autocmd_no_enter INIT(= FALSE); // Buf/WinEnter autocmds disabled +EXTERN int autocmd_no_leave INIT(= FALSE); // Buf/WinLeave autocmds disabled +EXTERN int tabpage_move_disallowed INIT(= FALSE); // moving tabpages around disallowed EXTERN int modified_was_set; // did ":set modified" EXTERN int did_filetype INIT(= FALSE); // FileType event found |