GNU bug report logs - #77140
[PATCH] Strip text properties on recentf entries

Previous Next

Full log

View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Ship Mints <shipmints <at> gmail.com>
Subject: bug#77140: closed (Re: bug#77140: [PATCH] Strip text properties
 on recentf entries)
Date: Sat, 19 Apr 2025 14:12:02 +0000

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

Your bug report

#77140: [PATCH] Strip text properties on recentf entries

which was filed against the emacs package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to 77140 <at> debbugs.gnu.org.

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

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

From: Eli Zaretskii <eliz <at> gnu.org>
To: Ship Mints <shipmints <at> gmail.com>
Cc: 77140-done <at> debbugs.gnu.org, stefankangas <at> gmail.com
Subject: Re: bug#77140: [PATCH] Strip text properties on recentf entries
Date: Sat, 19 Apr 2025 17:10:55 +0300

> From: Ship Mints <shipmints <at> gmail.com>
> Date: Wed, 9 Apr 2025 16:14:04 -0400
> Cc: stefankangas <at> gmail.com, 77140 <at> debbugs.gnu.org
> 
> On Fri, Mar 21, 2025 at 11:13 AM Ship Mints <shipmints <at> gmail.com> wrote:
> 
>  On Fri, Mar 21, 2025 at 11:07 AM Eli Zaretskii <eliz <at> gnu.org> wrote:
> 
>  > From: Ship Mints <shipmints <at> gmail.com>
>  > Date: Fri, 21 Mar 2025 10:02:56 -0400
>  > Cc: stefankangas <at> gmail.com, 77140 <at> debbugs.gnu.org
>  > 
>  >  Thanks, but please make the lines in the log message shorter (see
>  >  CONTRIBUTE for guidance).  This includes both the heading and the rest
>  >  of the log message.
>  > 
>  > This commit log entry summary is 64 chars (less than the 79 max target) and the rest of the log
>  is filled to 72
>  > (.dir-locals.el is in effect), so I'm missing what you are suggesting.
> 
>  The log message is indented with a TAB when we generate the ChangeLog
>  file from the Git log, so the 72-character lines will be 81, more than
>  we'd like to see in ChangeLog.  CONTRIBUTE says "Lines in ChangeLog
>  entries should preferably be not longer than 63 characters" for that
>  reason.  Please try not to exceed this limit, unless you really must
>  (like if you need to quote a long URL there).
> 
>  Got it.  I've adjusted my 'git-commit-mode' 'fill-column' accordingly.  Revised patch attached.
> 
> Possible to install this patch?  I've attached the same copy of the patch from the above so it's easy to find.

Thanks, now installed, and closing the bug.

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

From: Ship Mints <shipmints <at> gmail.com>
To: bug-gnu-emacs <at> gnu.org
Subject: [PATCH] Strip text properties on recentf entries
Date: Thu, 20 Mar 2025 12:56:25 -0400

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

Text properties are added by common completion frameworks, and serve no
purpose when stored in 'recentf-save-file', waste space, and slow loading
the file.

I did not add this to NEWS as I consider this non-user visible.  Happy to
add this if the maintainers disagree.

I've been running recentf with 'substring-no-properties' for ages and
others should benefit, by default.

-Stephane

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

[0001-Strip-text-properties-on-recentf-entries.patch (application/octet-stream, attachment)]

Previous Next

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