From: Philipp Stephani <phst@google.com>
Date: Sat, 1 Oct 2016 18:13:53 +0000 (+0200)
Subject: Use a simple keyword for a non-nil argument
X-Git-Tag: emacs-26.0.90~1521
X-Git-Url: http://git.eshelyaron.com/gitweb/?a=commitdiff_plain;h=08b386db1f4212ce90f726442fcf356885fb31ae;p=emacs.git

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 b13621a5c50..d3b6353d2a2 100644
--- 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))))
 
 
 ;; Frame maximization/fullscreen