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)
100% right. I think this silly typo/thinko was me. I could submit a patch but I don't have commit access. It would be faster for this one-liner if someone else patched.