GNU bug report logs - #44108
calc windows should be marked as dedicated

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 44108 in the body.
You can then email your comments to 44108 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Boruch Baum <boruch_baum <at> gmx.com>
To: Emacs Bug Reporting <bug-gnu-emacs <at> gnu.org>
Subject: calc windows should be marked as dedicated
Date: Tue, 20 Oct 2020 19:02:19 -0400

Windows that display calc buffers (ie. the stack buffer and the trail
buffer) are typically small-sized single-use windows, yet they are not
marked as dedicated. The consequence is that functions that perform
`pop-to-buffer' or `display-buffer' can use one of them as targets,
which will always be undesirable. A simple example, the one that
prompted this bug report, is when I was in a calc buffer and issued calc
command "u v" to display a 'units description' buffer. This is a long
and wide buffer, yet was displayed in the short and narrow calc trail
window. I vaguely recall having this also happen in the past with help
buffers, but was only motivated now to submit the report.

The solution is that when the calc and calc trail buffers are created,
they should be marked (set-window-dedicated-p)

--
hkp://keys.gnupg.net
CA45 09B5 5351 7C11 A9D1  7'286 0036 9E45 1595 8BC0

Message #8 received at 44108 <at> debbugs.gnu.org (full text, mbox):

From: Boruch Baum <boruch_baum <at> gmx.com>
To: 44108 <at> debbugs.gnu.org
Subject: Needs to be performed whenever windows are created
Date: Tue, 20 Oct 2020 19:09:13 -0400

I need to elaborate on my initial report: people who use calc a lot will
be closing and re-opening windows used for the same calc buffers, so the
(set-window-dedicated-p) operation needs to be done at each window
creation event for calc and calc-trail windows, not at buffer creation
events.

--
hkp://keys.gnupg.net
CA45 09B5 5351 7C11 A9D1  7286 0036 9E45 1595 8BC0

Message #11 received at 44108 <at> debbugs.gnu.org (full text, mbox):

From: Boruch Baum <boruch_baum <at> gmx.com>
To: 44108 <at> debbugs.gnu.org
Subject: Patch for this bug
Date: Thu, 22 Oct 2020 03:45:09 -0400

[Message part 1 (text/plain, inline)]

Attached.

--
hkp://keys.gnupg.net
CA45 09B5 5351 7C11 A9D1  7286 0036 9E45 1595 8BC0

[calc-dedicated-windows.patch (text/x-diff, attachment)]

Message #16 received at 44108 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Boruch Baum <boruch_baum <at> gmx.com>
Cc: 44108 <at> debbugs.gnu.org
Subject: Re: bug#44108: Patch for this bug
Date: Sat, 31 Oct 2020 09:54:16 +0200

> Date: Thu, 22 Oct 2020 03:45:09 -0400
> From: Boruch Baum <boruch_baum <at> gmx.com>
> 
> diff --git a/calc.el b/calc.el
> index 4bebd5f..90dcce6 100644
> --- a/calc.el
> +++ b/calc.el
> @@ -1472,6 +1472,7 @@ commands given here will actually operate on the *Calculator* stack."
>        (and (windowp full-display)
>             (window-point full-display)
>             (select-window full-display))
> +      (set-window-dedicated-p nil t)
>        (calc-check-defines)
>        (when (and calc-said-hello interactive)
>          (sit-for 2)
> @@ -2136,7 +2137,8 @@ the United States."
>                (if calc-trail-window-hook
>                    (run-hooks 'calc-trail-window-hook)
>                  (let ((w (split-window nil (/ (* (window-width) 2) 3) t)))
> -                  (set-window-buffer w calc-trail-buffer)))
> +                  (set-window-buffer w calc-trail-buffer)
> +                  (set-window-dedicated-p w t)))
>                (calc-wrapper
>                 (setq overlay-arrow-string calc-trail-overlay
>                       overlay-arrow-position calc-trail-pointer)

Thanks.

Would people who routinely use Calc please chime in and comment on
this?  Could this have some adverse consequences in some use cases?
Should we have a user option for controlling this behavior?

Message #19 received at 44108 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: boruch_baum <at> gmx.com
Cc: 44108 <at> debbugs.gnu.org
Subject: Re: bug#44108: Patch for this bug
Date: Sat, 07 Nov 2020 10:17:15 +0200

Ping!  Could someone please chime in?
If no comments are heard, I will push this change conditioned on a new
defcustom, in one week's time.

> Date: Sat, 31 Oct 2020 09:54:16 +0200
> From: Eli Zaretskii <eliz <at> gnu.org>
> Cc: 44108 <at> debbugs.gnu.org
> 
> > Date: Thu, 22 Oct 2020 03:45:09 -0400
> > From: Boruch Baum <boruch_baum <at> gmx.com>
> > 
> > diff --git a/calc.el b/calc.el
> > index 4bebd5f..90dcce6 100644
> > --- a/calc.el
> > +++ b/calc.el
> > @@ -1472,6 +1472,7 @@ commands given here will actually operate on the *Calculator* stack."
> >        (and (windowp full-display)
> >             (window-point full-display)
> >             (select-window full-display))
> > +      (set-window-dedicated-p nil t)
> >        (calc-check-defines)
> >        (when (and calc-said-hello interactive)
> >          (sit-for 2)
> > @@ -2136,7 +2137,8 @@ the United States."
> >                (if calc-trail-window-hook
> >                    (run-hooks 'calc-trail-window-hook)
> >                  (let ((w (split-window nil (/ (* (window-width) 2) 3) t)))
> > -                  (set-window-buffer w calc-trail-buffer)))
> > +                  (set-window-buffer w calc-trail-buffer)
> > +                  (set-window-dedicated-p w t)))
> >                (calc-wrapper
> >                 (setq overlay-arrow-string calc-trail-overlay
> >                       overlay-arrow-position calc-trail-pointer)
> 
> Thanks.
> 
> Would people who routinely use Calc please chime in and comment on
> this?  Could this have some adverse consequences in some use cases?
> Should we have a user option for controlling this behavior?
> 
> 
> 
> 

Message #22 received at 44108 <at> debbugs.gnu.org (full text, mbox):

From: martin rudalics <rudalics <at> gmx.at>
To: Eli Zaretskii <eliz <at> gnu.org>, boruch_baum <at> gmx.com
Cc: 44108 <at> debbugs.gnu.org
Subject: Re: bug#44108: Patch for this bug
Date: Sat, 7 Nov 2020 09:59:21 +0100

> Ping!  Could someone please chime in?
> If no comments are heard, I will push this change conditioned on a new
> defcustom, in one week's time.

IMHO _any_ use of dedicated windows in Emacs should be optional so a
defcustom is certainly a good idea.

Note that 'display-buffer' already processes a 'dedicated' action alist
entry so if 'calc' used it in this context, the option would already be
available now.

martin

Message #27 received at 44108-done <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: boruch_baum <at> gmx.com
Cc: 44108-done <at> debbugs.gnu.org
Subject: Re: bug#44108: Patch for this bug
Date: Sat, 14 Nov 2020 15:04:40 +0200

> Date: Sat, 07 Nov 2020 10:17:15 +0200
> From: Eli Zaretskii <eliz <at> gnu.org>
> Cc: 44108 <at> debbugs.gnu.org
> 
> Ping!  Could someone please chime in?
> If no comments are heard, I will push this change conditioned on a new
> defcustom, in one week's time.

Done.

Previous Next

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