GNU bug report logs - #68235
29.1.90; Switching tabs stops following process output in selected window

Previous Next

Full log

View this message in rfc822 format

From: Juri Linkov <juri <at> linkov.net>
To: martin rudalics <rudalics <at> gmx.at>
Cc: 68235 <at> debbugs.gnu.org
Subject: bug#68235: 29.1.90; Switching tabs stops following process output in selected window
Date: Wed, 06 Mar 2024 19:57:53 +0200

>> So when most of the time this list of kept windows will be empty,
>> then maybe better to not call the hook at all?
>
> But earlier you said "Running the hook with an empty list of windows
> makes sense as well."  So it's up to you.

We could leave it called on every set-window-configuration
even with an empty window list, as a general post-hook.

BTW, I didn't test yet what happens when
window-kept-windows-functions is non-nil but does nothing.
Here is what happens:

  Error muted by safe_call: (get-scratch-buffer-create)
  signaled (wrong-type-argument window-live-p #<window 7>)

But maybe this is ok.

>> 3. Very often the message inserted by the patch that I posted
>> are quite useless because they look like this:
>>
>>     This window displayed the buffer #<killed buffer>.
>>
>> This would be much more informative:
>>
>>     This window displayed the buffer *Help*.
>>
>> Maybe 'buffer-last-name' could help to achieve this?
>
> I tried to implement it.  Tested with
>
> (let ((buffer (get-buffer-create "*foo*")))
>   (y-or-n-p (format "current %s last %s"
> 		    (buffer-name buffer) (buffer-last-name buffer)))
>   (with-current-buffer buffer
>     (rename-buffer "*bar*"))
>   (y-or-n-p (format "current %s last %s"
> 		    (buffer-name buffer) (buffer-last-name buffer)))
>   (kill-buffer buffer)
>   (y-or-n-p (format "current %s last %s"
> 		    (buffer-name buffer) (buffer-last-name buffer))))
> [...]
> Have a look at the attached patch.

Thanks.  I tested, and everything works nicely.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.