GNU bug report logs - #76123
[PATCH] Eliminate savehist duplicated symbols

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#76123: closed (Re: bug#76123: [PATCH] Eliminate savehist
 duplicated symbols)
Date: Sat, 08 Feb 2025 12:09:02 +0000

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

Your bug report

#76123: [PATCH] Eliminate savehist duplicated symbols

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 76123 <at> debbugs.gnu.org.

-- 
76123: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=76123
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: 76123-done <at> debbugs.gnu.org, stefankangas <at> gmail.com
Subject: Re: bug#76123: [PATCH] Eliminate savehist duplicated symbols
Date: Sat, 08 Feb 2025 14:07:58 +0200

> From: Ship Mints <shipmints <at> gmail.com>
> Date: Sat, 8 Feb 2025 05:38:47 -0500
> Cc: stefankangas <at> gmail.com, 76123 <at> debbugs.gnu.org
> 
> Surely. Revised patch attached with a NEWS entry.

Thanks, installed on master, and closing the bug.

> I did not yet combine the two tests (the code is nearly duplicated) if that's what you think should be done so
> if you want me to do that, I can today.

I see no need.

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

From: Ship Mints <shipmints <at> gmail.com>
To: bug-gnu-emacs <at> gnu.org
Subject: [PATCH] Eliminate savehist duplicated symbols
Date: Fri, 7 Feb 2025 15:05:40 -0500

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

This patch avoids savehist saving symbols duplicated between
'savehist-minibuffer-history-variables' and
'savehist-additional-variables'. I've accumulated symbols on
'savehist-additional-variables' that wind up, over time, in the master
minibuffer list documented as being dynamic.

  The contents of this variable is built while Emacs is running,
  and saved along with minibuffer history.

savehist had no defensive mechanism until now. The patch should do no harm
to users and will transparently save some resources.

TIA,

-Stephane

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

[0001-Eliminate-savehist-duplicated-symbols.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.