GNU bug report logs - #41108
27.0.91; `register-read-with-preview' delete pre existing window

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 41108 in the body.
You can then email your comments to 41108 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: Héctor Enríquez Ramón <hector.e.r <at> gmail.com>
To: bug-gnu-emacs <at> gnu.org
Subject: 27.0.91; `register-read-with-preview' delete pre existing window
Date: Wed, 6 May 2020 13:21:42 +0200

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

 Before `window-configuration-to-register'

 +-------------------+-----------------+
 |                   |                 |
 |                   |                 |
 |                   |                 |
 |                   |                 |
 +-------------------+-----------------+

 After `window-configuration-to-register'

 +-------------------------------------+
 |                                     |
 |                                     |
 |                                     |
 |                                     |
 +-------------------------------------+

 Then last (wrong) window configuration is saved to register.

 ```
 (defun register-read-with-preview (prompt)
 ...
      (let ((w (get-buffer-window buffer)))
        (and (window-live-p w) (delete-window w)))   <=== ### Don't check
 pre existing window ###
      (and (get-buffer buffer) (kill-buffer buffer)))))
 ```

 Thanks. Best regards.


 In GNU Emacs 27.0.91 (build 4, x86_64-pc-linux-gnu, cairo version 1.16.0)
 of 2020-05-06 built on Ubuntu-18
 Repository revision: f8e6cd11b34320999aa72886ece3f32b0a8ad897
 Repository branch: emacs-27
 Windowing system distributor 'The X.Org Foundation', version 11.0.12005000
 System Description: Ubuntu 19.10

 Configured using:
 'configure --with-cairo --with-x-toolkit=no CFLAGS=-O3'

 Configured features:
 XPM JPEG TIFF GIF PNG CAIRO SOUND GPM DBUS GSETTINGS GLIB NOTIFY INOTIFY
 ACL LIBSELINUX GNUTLS LIBXML2 FREETYPE HARFBUZZ ZLIB OLDXMENU X11 XDBE
 XIM MODULES THREADS PDUMPER GMP

 Important settings:
  value of $LC_ALL:
  value of $LANG: es_ES.UTF-8
  locale-coding-system: utf-8
Date: Wed, 06 May 2020 13:11:21 +0200
Message-ID:
<87zhalgvxi.fsf <at> Ubuntu-18.i-did-not-set--mail-host-address--so-tickle-me>

[Message part 2 (text/html, inline)]

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

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: Héctor Enríquez Ramón <hector.e.r <at> gmail.com>
Cc: 41108 <at> debbugs.gnu.org
Subject: Re: bug#41108: 27.0.91; `register-read-with-preview' delete pre
 existing window
Date: Thu, 22 Oct 2020 16:18:36 +0200

Héctor Enríquez Ramón <hector.e.r <at> gmail.com> writes:

>  Before `window-configuration-to-register'
>
>  +-------------------+-----------------+
>  |                   |                 |
>  |                   |                 |
>  |                   |                 |
>  |                   |                 |
>  +-------------------+-----------------+
>
>  After `window-configuration-to-register'
>
>  +-------------------------------------+
>  |                                     |
>  |                                     |
>  |                                     |
>  |                                     |
>  +-------------------------------------+
>
>  Then last (wrong) window configuration is saved to register.

I'm unable to reproduce this in Emacs 27.2, but I can't see any obvious
changes in this code, either.

Can you give a recipe to reproduce this bug, starting from "emacs -Q"?

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no

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

From: Héctor Enríquez Ramón <hector.e.r <at> gmail.com>
To: Lars Ingebrigtsen <larsi <at> gnus.org>
Cc: 41108 <at> debbugs.gnu.org
Subject: Re: bug#41108: 27.0.91; `register-read-with-preview' delete pre
 existing window
Date: Thu, 22 Oct 2020 19:45:30 +0200

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

Hi Lars.

It works fine in Emacs branch emacs-27 (last commit).

Thank you very much. Best regards.

On Thu, 22 Oct 2020 at 16:18, Lars Ingebrigtsen <larsi <at> gnus.org> wrote:

> Héctor Enríquez Ramón <hector.e.r <at> gmail.com> writes:
>
> >  Before `window-configuration-to-register'
> >
> >  +-------------------+-----------------+
> >  |                   |                 |
> >  |                   |                 |
> >  |                   |                 |
> >  |                   |                 |
> >  +-------------------+-----------------+
> >
> >  After `window-configuration-to-register'
> >
> >  +-------------------------------------+
> >  |                                     |
> >  |                                     |
> >  |                                     |
> >  |                                     |
> >  +-------------------------------------+
> >
> >  Then last (wrong) window configuration is saved to register.
>
> I'm unable to reproduce this in Emacs 27.2, but I can't see any obvious
> changes in this code, either.
>
> Can you give a recipe to reproduce this bug, starting from "emacs -Q"?
>
> --
> (domestic pets only, the antidote for overdose, milk.)
>    bloggy blog: http://lars.ingebrigtsen.no
>

[Message part 2 (text/html, inline)]

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

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: Héctor Enríquez Ramón <hector.e.r <at> gmail.com>
Cc: 41108 <at> debbugs.gnu.org
Subject: Re: bug#41108: 27.0.91; `register-read-with-preview' delete pre
 existing window
Date: Fri, 23 Oct 2020 12:50:02 +0200

Héctor Enríquez Ramón <hector.e.r <at> gmail.com> writes:

> It works fine in Emacs branch emacs-27 (last commit).

Thanks for checking; closing the bug report.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no

Previous Next

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