patch 9.1.0128: win_gotoid() may abort even when not switching a window
Problem: win_gotoid() checks for textlock and other things when switching
to a window that is already current (after v9.1.0119)
Solution: return early with success when attempting to switch to curwin
(Sean Dewar)
Other potential causes of E565 from win_gotoid after v9.1.0119 should be
correct. Plugins can consider using win_execute() instead if they wish to
temporarily switch windows during textlock.
fixes: #14073
closes: #14074
Signed-off-by: Sean Dewar <6256228+seandewar@users.noreply.github.com>
Signed-off-by: Christian Brabandt <cb@256bit.org>
diff --git a/src/evalwindow.c b/src/evalwindow.c
index 00081d4..f9b1c12 100644
--- a/src/evalwindow.c
+++ b/src/evalwindow.c
@@ -824,6 +824,13 @@
return;
id = tv_get_number(&argvars[0]);
+ if (curwin->w_id == id)
+ {
+ // Nothing to do.
+ rettv->vval.v_number = 1;
+ return;
+ }
+
if (text_or_buf_locked())
return;
#if defined(FEAT_PROP_POPUP) && defined(FEAT_TERMINAL)