bug-gnu-emacs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#63967: 28.2; switch-to-buffer in normal window fails if minibuffer w


From: martin rudalics
Subject: bug#63967: 28.2; switch-to-buffer in normal window fails if minibuffer window is active
Date: Sat, 10 Jun 2023 08:52:57 +0200

> diff --git a/lisp/window.el b/lisp/window.el
> index a11b1a5..6777944 100644
> --- a/lisp/window.el
> +++ b/lisp/window.el
> @@ -8941,7 +8941,9 @@ switch-to-buffer
>                       "Cannot switch buffers in a dedicated window")))
>                   ('pop nil)
>                   (_ (set-window-dedicated-p nil nil) 
'force-same-window)))))))
> -     (list (read-buffer-to-switch "Switch to buffer: ") nil 
force-same-window)))
> +     (save-selected-window
> +       (list
> +        (read-buffer-to-switch "Switch to buffer: ") nil 
force-same-window))))
>     (let ((buffer (window-normalize-buffer-to-switch-to buffer-or-name))
>           (set-window-start-and-point (not 
switch-to-buffer-obey-display-actions)))
>       (cond

That wouldn't help in all other use cases of read_minibuf where the user
will be thrown back to the minibuffer window as well.  I'd rather try (the
still timid)

-static void minibuffer_unwind (void);
+static void minibuffer_unwind (Lisp_Object);
...
-  record_unwind_protect_void (minibuffer_unwind);
+  record_unwind_protect (minibuffer_unwind, selected_window);
...
-minibuffer_unwind (void)
+minibuffer_unwind (Lisp_Object old_selected_window)
...
+         if (!EQ (old_selected_window, FRAME_SELECTED_WINDOW (f)))
+           Fset_frame_selected_window (exp_MB_frame, window, Qnil);

since the last line seems to suggest that exp_MB_frame should not be the
selected frame.

martin





reply via email to

[Prev in Thread] Current Thread [Next in Thread]