bug#34179: 27.0.50; message hangs when buffer with process visible

[martin rudalics <rudalics@gmx.at>]

[-- Attachment #1: Type: text/plain, Size: 2085 bytes --]

 >> Here's the Lisp backtrace:
 >>
 >>    "select-window" (0x82a508)
 >>    "window-max-chars-per-line" (0x82aae0)
 >>    "window-adjust-process-window-size" (0x82b0a0)
 >>    "window-adjust-process-window-size-smallest" (0x82b618)
 >>    "window--adjust-process-windows" (0x82bbc0)
 >>    "redisplay_internal (C function)" (0x0)
 >>    "message" (0x82cdb0)
 >>    "progn" (0x82cff8)
 >>    "if" (0x82d198)
 >>    "let*" (0x82d3c8)
 >>    "catch" (0x82d608)
 >>    "fracy-display-frame-list" (0x82d7c0)
 >>    "if" (0x82dac8)
 >>    "fracy-flash-window" (0x82dc80)
 >>    "let" (0x82e018)
 >>    "ad-Advice-handle-switch-frame" (0x82e4a0)
 >>    "apply" (0x82e498)
 >>    "handle-switch-frame" (0x82ebe0)
 >>    "funcall-interactively" (0x82ebd8)
 >>    "call-interactively" (0x82f030)
 >>    "command-execute" (0x82f628)
 >>
 >> The C portion of the backtrace below selec t-window is this:
 >>
 >>    #0  redisplay_other_windows () at xdisp.c:597
 >>    #1  0x010aa09e in select_window (window=XIL(0xa000000006919aa0),
 >>        norecord=XIL(0x3d36a14), inhibit_point_swap=false) at window.c:540
 >>    #2  0x010aa62a in Fselect_window (window=XIL(0xa000000006919aa0),
 >>        norecord=XIL(0x3d36a14)) at window.c:628
 >
 > To clarify: these backtraces show where windows_or_buffers_changed is
 > being set to the value of 2, which causes us to retry redisplay.  It
 > is a direct consequence of the fact that window-max-chars-per-line
 > calls select-window.

If I replace the line

   (with-selected-window (window-normalize-window window t)

of 'window-max-chars-per-line' by the line

   (with-current-buffer (window-buffer window)

the problem persists.  So it is not the 'select-window' call per se
that causes the trouble.  The problem is with having that separate
frame that gets its echo area resized.  I'm not sure what causes it
but I suppose it's 'window-configuration-change-hook' itself that
selects that frame for running the hook.

Using 'window-size-change-functions' (patch attached) instead of
'window-configuration-change-hook' fixes the problem here.

martin

[-- Attachment #2: window--adjust-process-windows.diff --]
[-- Type: text/plain, Size: 784 bytes --]

diff --git a/lisp/window.el b/lisp/window.el
index e806677..3a7f7dc 100644
--- a/lisp/window.el
+++ b/lisp/window.el
@@ -9610,7 +9610,7 @@ window--process-window-list
          1 t))
     process-windows))
 
-(defun window--adjust-process-windows ()
+(defun window--adjust-process-windows (_frame_or_window)
   "Update process window sizes to match the current window configuration."
   (when (fboundp 'process-list)
     (dolist (procwin (window--process-window-list))
@@ -9624,7 +9624,7 @@ window--adjust-process-windows
               (when size
                 (set-process-window-size process (cdr size) (car size))))))))))
 
-(add-hook 'window-configuration-change-hook 'window--adjust-process-windows)
+(add-hook 'window-size-change-functions 'window--adjust-process-windows)