patch 8.1.1425: win_execute() does not set window pointers properly
Problem: Win_execute() does not set window pointers properly.
Solution: Use switch_win_noblock(). Also execute autocommands in a popup
window.
diff --git a/src/evalfunc.c b/src/evalfunc.c
index 7753fc6..d1e89eb 100644
--- a/src/evalfunc.c
+++ b/src/evalfunc.c
@@ -6116,19 +6116,18 @@
{
int id = (int)tv_get_number(argvars);
win_T *wp = win_id2wp(id);
- win_T *save_curwin = curwin;
+ win_T *save_curwin;
+ tabpage_T *save_curtab;
if (wp != NULL)
{
- curwin = wp;
- curbuf = curwin->w_buffer;
- check_cursor();
- execute_common(argvars, rettv, 1);
- if (win_valid(save_curwin))
+ if (switch_win_noblock(&save_curwin, &save_curtab, wp, curtab, TRUE)
+ == OK)
{
- curwin = save_curwin;
- curbuf = curwin->w_buffer;
+ check_cursor();
+ execute_common(argvars, rettv, 1);
}
+ restore_win_noblock(save_curwin, save_curtab, TRUE);
}
}