From: Eli Zaretskii <eliz@gnu.org>
Date: Fri, 1 Jun 2018 12:34:30 +0000 (+0300)
Subject: Add commentary for subtle aspect of frame.el
X-Git-Tag: emacs-26.1.90~427
X-Git-Url: http://git.eshelyaron.com/gitweb/?a=commitdiff_plain;h=e5471b2381e885d5d214bfa09ab0c35275fc6048;p=emacs.git

Add commentary for subtle aspect of frame.el

* lisp/frame.el: Explain why we use symbol-function when adding
watchers for certain variables that need to trigger redisplay.
---

diff --git a/lisp/frame.el b/lisp/frame.el
index 447413b3256..29c31f41cb1 100644
--- a/lisp/frame.el
+++ b/lisp/frame.el
@@ -2475,6 +2475,9 @@ See also `toggle-frame-maximized'."
 ;; F5 then produces the correct effect, the variable doesn't need
 ;; to be in this list; otherwise, it does.
 (mapc (lambda (var)
+        ;; Using symbol-function here tells the watcher machinery to
+        ;; call the C function set-buffer-redisplay directly, thus
+        ;; avoiding a potential GC.
         (add-variable-watcher var (symbol-function 'set-buffer-redisplay)))
       '(line-spacing
         overline-margin