GNU bug report logs - #78820
31.0.50; savehist timer does not run repeatedly

Previous Next

Full log

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

From: Stéphane Marks <shipmints <at> gmail.com>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: Daniel Mendler <mail <at> daniel-mendler.de>, 78820 <at> debbugs.gnu.org,
 monnier <at> iro.umontreal.ca
Subject: Re: bug#78820: 31.0.50; savehist timer does not run repeatedly
Date: Wed, 18 Jun 2025 08:12:39 -0400

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

On Wed, Jun 18, 2025 at 7:55 AM Eli Zaretskii <eliz <at> gnu.org> wrote:

> > Cc: "Stefan Monnier" <monnier <at> iro.umontreal.ca>
> > Date: Wed, 18 Jun 2025 02:23:14 +0200
> > From:  Daniel Mendler via "Bug reports for GNU Emacs,
> >  the Swiss army knife of text editors" <bug-gnu-emacs <at> gnu.org>
> >
> > It seems that the savehist timer does not run repeatedly. run-with-timer
> > is called with repeat argument t.
> >
> > (run-with-timer savehist-autosave-interval t #'savehist-autosave)
> >
> > If I understand correctly, the correct call should look like this:
> >
> > (run-with-timer t savehist-autosave-interval #'savehist-autosave)
> > (run-with-timer savehist-autosave-interval savehist-autosave-interval
> #'savehist-autosave)
>
> Why do you think so?  The calling sequence of run-with-timer is this:
>
>   (run-with-timer SECS REPEAT FUNCTION &rest ARGS)
>
> So what savehist.el does appears to be correct.  Or what am I missing?
>

(defun savehist--manage-timer ()
;; should be
   (run-with-timer savehist-autosave-interval
                   savehist-autosave-interval #'savehist-autosave))
;; not silly t
   (run-with-timer savehist-autosave-interval t #'savehist-autosave))

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

Previous Next

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