GNU bug report logs - #45947
28.0.50; comment-dwim and undo in active region

Previous Next

To reply to this bug, email your comments to 45947 AT debbugs.gnu.org.

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

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

From: Bodertz <bodertz <at> gmail.com>
To: bug-gnu-emacs <at> gnu.org
Subject: 28.0.50; comment-dwim and undo in active region
Date: Sun, 17 Jan 2021 22:53:39 -0700

Undoing with an active region does not always work as expected if
`comment-dwim' is used.  From emacs -Q:

1. Create a new buffer (C-x b new-buffer RET)

2. Insert A, B, and C on seperate lines as below
   ('*' represents point)

A
B
C*

3. Create an active region over the last two lines (C-SPC C-p C-a)
4. Use comment-dwim (M-; ; RET)
5. Create an active region over the last two lines (C-SPC M->)
6. Undo (C-/):

A
; *
; C

7. Undo again (C-/):

A
;*
; C

8. Undo again (C-/):

A
*
; C

In GNU Emacs 28.0.50 (build 1, x86_64-pc-linux-gnu, GTK+ Version 3.24.23, cairo version 1.16.0)
Windowing system distributor 'System Description: openSUSE Tumbleweed

Configured using:
 'configure
 CONFIG_SHELL=/gnu/store/pwcp239kjf7lnj5i4lkdzcfcxwcfyk72-bash-minimal-5.0.16/bin/bash
 SHELL=/gnu/store/pwcp239kjf7lnj5i4lkdzcfcxwcfyk72-bash-minimal-5.0.16/bin/bash
 --prefix=/gnu/store/k327dvhl6d897r3jz7gyhdyf0rdvdarh-emacs-next-pgtk-28.0.50-0.d46a223
 --enable-fast-install --with-pgtk --with-xwidgets --with-modules
 --with-cairo --disable-build-details'

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

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: Bodertz <bodertz <at> gmail.com>
Cc: 45947 <at> debbugs.gnu.org
Subject: Re: bug#45947: 28.0.50; comment-dwim and undo in active region
Date: Tue, 07 Jun 2022 16:14:47 +0200

Bodertz <bodertz <at> gmail.com> writes:

> Undoing with an active region does not always work as expected if
> `comment-dwim' is used.  From emacs -Q:
>
> 1. Create a new buffer (C-x b new-buffer RET)
>
> 2. Insert A, B, and C on seperate lines as below
>    ('*' represents point)
>
> A
> B
> C*
>
> 3. Create an active region over the last two lines (C-SPC C-p C-a)
> 4. Use comment-dwim (M-; ; RET)
> 5. Create an active region over the last two lines (C-SPC M->)
> 6. Undo (C-/):
>
> A
> ; *
> ; C

(I'm going through old bug reports that unfortunately weren't resolved
at the time.)

I can reproduce this in Emacs 29, too.  I wonder whether it's related to
bug#21523 -- it, too, mis-computes the active region at the edges and
includes the wrong thing.

-- 
(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.