Subject: bug#75834: closed (Re: bug#75834: [PATCH] Add missing custom :set to savehist-autosave-interval)

From: [email-masked] (GNU bug Tracking System)
Date: Sat, 01 Feb 2025 20:10:02 +0000

Your bug report

#75834: [PATCH] Add missing custom :set to savehist-autosave-interval

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 [email-masked].

-- 
75834: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=75834
GNU Bug Tracking System
Contact [email-masked] with problems

Subject: Re: bug#75834: [PATCH] Add missing custom :set to savehist-autosave-interval
From: Eli Zaretskii <[email-masked]>
To: Ship Mints <[email-masked]>
Cc: [email-masked]
Date: Sat, 01 Feb 2025 22:09:30 +0200
List-Id: <debbugs-submit.debbugs.gnu.org>

> From: Ship Mints <[email-masked]>
> Date: Sat, 25 Jan 2025 13:19:11 -0500
> 
> * lisp/savehist.el (savehist-autosave-interval):
> Correctly reset 'savehist-timer' when savehist-autosave-interval changes
> via setopt or a Customize command.

Thanks, installed on the master branch, and closing the bug.

Subject: [PATCH] Add missing custom :set to savehist-autosave-interval
From: Ship Mints <[email-masked]>
To: [email-masked]
Date: Sat, 25 Jan 2025 13:19:11 -0500
List-Id: <debbugs-submit.debbugs.gnu.org>

* lisp/savehist.el (savehist-autosave-interval):
Correctly reset 'savehist-timer' when savehist-autosave-interval changes
via setopt or a Customize command.

An open question is should the timer be an idle timer or not. Currently,
the timer is a regular timer. I'd prefer an idle timer. Not sure how many
people would care if this changed.