GNU bug report logs - #78262
30.1; comint-delchar-or-maybe-eof doesn't respect delete-active-region

Previous Next

Package: emacs;

Reported by: Sebastián Monía <sebastian <at> sebasmonia.com>

Date: Mon, 5 May 2025 14:45:02 UTC

Severity: normal

Merged with 73527

Found in versions 30.1, 30.0.90

Fixed in version 31.0.50

Done: Juri Linkov <juri <at> linkov.net>

Bug is archived. No further changes may be made.

Full log


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

From: Stefan Monnier <monnier <at> iro.umontreal.ca>
To: Juri Linkov <juri <at> linkov.net>
Cc: sebastian <at> sebasmonia.com, Eli Zaretskii <eliz <at> gnu.org>,
 78262 <at> debbugs.gnu.org
Subject: Re: bug#78262: 30.1; comint-delchar-or-maybe-eof doesn't respect
 delete-active-region
Date: Mon, 12 May 2025 00:28:00 -0400
>>> PS: this still requires
>>>
>>>   (put 'comint-delchar-or-maybe-eof 'delete-selection 'supersede)
>>>
>>> that could be added later.
>>
>> Hmm... sorry my `delete-selection`-fu is rusty, what would this
>> do/fix, again?
>
> This is a copy from delsel.el
>
>   ;; delete-backward-char and delete-forward-char already delete the selection by
>   ;; default, but not delete-char.
>   (put 'delete-char 'delete-selection 'supersede)
>
> that will do the same for 'comint-delchar-or-maybe-eof'
> that uses 'delete-char' by default.

Ah.  I consider this part of delsel.el to be obsolete (made obsolete
back in Emacs-24 by the `delete-active-region` variable).
If `delsel.el` wants to make `C-d` delete the active region, it should
preferably do it by remapping `delete-char` to `delete-forward-char`, IMO.


        Stefan





This bug report was last modified 6 days ago.

Previous Next


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