GNU bug report logs - #79176
[PATCH] ; * lisp/indent-aux.el: Remove read-only text property

Previous Next

Full log

View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: tracker <at> debbugs.gnu.org
Subject: bug#79176: closed ([PATCH] ; * lisp/indent-aux.el: Remove
 read-only text property)
Date: Thu, 07 Aug 2025 13:15:01 +0000

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

Your message dated Thu, 07 Aug 2025 16:13:56 +0300
with message-id <86h5yjqnor.fsf <at> gnu.org>
and subject line Re: bug#79176: [PATCH] ; * lisp/indent-aux.el: Remove read-only text property
has caused the debbugs.gnu.org bug report #79176,
regarding [PATCH] ; * lisp/indent-aux.el: Remove read-only text property
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs <at> gnu.org.)


-- 
79176: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=79176
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems

[Message part 2 (message/rfc822, inline)]

From: Tony Zorman <mail <at> tony-zorman.com>
To: bug-gnu-emacs <at> gnu.org
Subject: [PATCH] ; * lisp/indent-aux.el: Remove read-only text property
Date: Tue, 05 Aug 2025 13:23:04 +0200

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

Tags: patch

Hi,

when one uses `kill-ring-deindent-mode' and selects read-only text, the
mode can't actually deindent it, which causes a lot of `beep' noise and
prevents the user from killing the selected text (or from any other
action, really). This patch fixes this by removing that particular
property before the text is inserted into the temporary buffer.

Thanks,
Tony

[0001-lisp-indent-aux.el-Remove-read-only-text-property.patch (text/patch, attachment)]

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

-- 
Tony Zorman | https://tony-zorman.com

[Message part 6 (message/rfc822, inline)]

From: Eli Zaretskii <eliz <at> gnu.org>
To: Tony Zorman <mail <at> tony-zorman.com>
Cc: 79176-done <at> debbugs.gnu.org
Subject: Re: bug#79176: [PATCH] ; * lisp/indent-aux.el: Remove read-only
 text property
Date: Thu, 07 Aug 2025 16:13:56 +0300

> From: Tony Zorman <mail <at> tony-zorman.com>
> Cc: 79176 <at> debbugs.gnu.org
> Date: Wed, 06 Aug 2025 08:12:58 +0200
> 
> > Thanks, but isn't it better to bind inhibit-read-only non-nil around
> > the code which modifies the text instead?  By default
> > yank-excluded-properties includes read-only, but what if someone
> > changes the value not to include read-only? won't they expect to see
> > the read-only property when the text is yanked?
> 
> Ah, I didn't know about yank-excluded-properties. You're right, in that
> case perhaps it's better to use inhibit-read-only; I've updated the
> patch accordingly.

Thanks, now installed on the master branch (after fixing the commit
log message, since it still talked about removing the property).

I'm therefore closing this bug.

Previous Next

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