Re: master e56e4b345a2: Use timer accessors instead of aref/aset

[Stefan Monnier <monnier@iro.umontreal.ca>]

> commit e56e4b345a25194bc7adb83523e8d886b718c9fa
> Author: Mattias Engdegård <mattiase@acm.org>
> Commit: Mattias Engdegård <mattiase@acm.org>
>
>     Use timer accessors instead of aref/aset
>     
>     * lisp/emacs-lisp/timer-list.el (list-timers):
>     * lisp/gnus/mail-source.el (mail-source-start-idle-timer):
>     * lisp/play/gamegrid.el (gamegrid-set-timer):
>     * lisp/progmodes/vhdl-mode.el (vhdl-run-when-idle):
>     * lisp/simple.el (analyze-text-conversion):
>     * lisp/time.el (display-time-event-handler):
>     Use timer accessors.
> diff --git a/lisp/progmodes/vhdl-mode.el b/lisp/progmodes/vhdl-mode.el
> index 144bfa944d3..22706be6bb5 100644
> --- a/lisp/progmodes/vhdl-mode.el
> +++ b/lisp/progmodes/vhdl-mode.el
> @@ -2341,7 +2341,7 @@ Ignore byte-compiler warnings you might see."
>    (if (fboundp 'start-itimer)
>        (start-itimer "vhdl-mode" function secs repeat t)
>      ;; explicitly activate timer (necessary when Emacs is already idle)
> -    (aset (run-with-idle-timer secs repeat function) 0 nil)))
> +    (setf (timer--triggered (run-with-idle-timer secs repeat function)) nil)))

`vhdl-mode.el` likes to be compatible with XEmacs, so this change might
be problematic.  I'm not sure if it's still important, but I'll also
note that, according to my reading of the `timer.el` code, the timer
objects returned by `run-with-idle-timer` already have their `triggered`
field set to nil.

I'm not sure if it's always been the case, but maybe we could make the
`aset` conditional on `(< emacs-major-version 30)` (or some older version
number once we investigate since when this is the case)?


        Stefan