Use a simple keyword for a non-nil argument

The second argument of `run-with-idle-timer' is Boolean, i.e. only nil
and non-nil values are distinguished.  Passing a number here is
confusing.  Pass a descriptive symbol instead.

* lisp/frame.el (blink-cursor-mode, blink-cursor-check): Use
:repeat symbol instead of number for second argument of
`run-with-idle-timer'

diff --git a/lisp/frame.el b/lisp/frame.el
index b13621a5c504efd139d2017f7e899551d8b88b6a..d3b6353d2a29b357786dad4992c61f531c576a23 100644 (file)
--- a/lisp/frame.el
+++ b/lisp/frame.el
@@ -2119,8 +2119,7 @@ This is done when a frame gets focus.  Blink timers may be stopped by
           ;; during command execution) if they set blink-cursor-delay
           ;; to a very small or even zero value.
           (run-with-idle-timer (max 0.2 blink-cursor-delay)
-                               blink-cursor-delay
-                               'blink-cursor-start))))
+                               :repeat #'blink-cursor-start))))
 
 (define-obsolete-variable-alias 'blink-cursor 'blink-cursor-mode "22.1")
 
@@ -2157,8 +2156,7 @@ terminals, cursor blinking is controlled by the terminal."
           ;; during command execution) if they set blink-cursor-delay
           ;; to a very small or even zero value.
           (run-with-idle-timer (max 0.2 blink-cursor-delay)
-                               blink-cursor-delay
-                               #'blink-cursor-start))))
+                               :repeat #'blink-cursor-start))))
 
 \f
 ;; Frame maximization/fullscreen