emacs: make the refresh functions more consistent
authorMark Walters <markwalters1009@gmail.com>
Sat, 15 Oct 2016 07:32:40 +0000 (08:32 +0100)
committerDavid Bremner <david@tethera.net>
Mon, 17 Oct 2016 12:01:52 +0000 (09:01 -0300)
The different refreshed functions were called differently: some were
called interactively and some were not. Make them all interactive.

emacs/notmuch-hello.el
emacs/notmuch-lib.el
emacs/notmuch.el

index d582bff7a4914588a7e6f62c89728139a4a93bba..089a19d72a989782a7ce809f183a3c8d0bd0437d 100644 (file)
@@ -607,6 +607,7 @@ with `notmuch-hello-query-counts'."
 (defun notmuch-hello-update (&optional no-display)
   "Update the current notmuch view."
   ;; Lazy - rebuild everything.
+  (interactive)
   (notmuch-hello no-display))
 
 (defun notmuch-hello-window-configuration-change ()
index b2cdace9e1262d860891ca436c88db1864a14d11..56dd366959227f78f78ddeaace5e3ccc204d144a 100644 (file)
@@ -417,10 +417,8 @@ of its command symbol."
   "Refresh the current buffer."
   (interactive)
   (when notmuch-buffer-refresh-function
-    (if (commandp notmuch-buffer-refresh-function)
-       ;; Pass prefix argument, etc.
-       (call-interactively notmuch-buffer-refresh-function)
-      (funcall notmuch-buffer-refresh-function))))
+    ;; Pass prefix argument, etc.
+    (call-interactively notmuch-buffer-refresh-function)))
 
 (defun notmuch-poll-and-refresh-this-buffer ()
   "Invoke `notmuch-poll' to import mail, then refresh the current buffer."
index 6c36ad8a9e4045182cda6a92b76fae2c4db52055..673811ce11eef791f846927095a706378d7500b3 100644 (file)
@@ -991,6 +991,7 @@ query string as the current search. If the current thread is in
 the new search results, then point will be placed on the same
 thread. Otherwise, point will be moved to attempt to be in the
 same relative position within the new buffer."
+  (interactive)
   (let ((target-line (line-number-at-pos))
        (oldest-first notmuch-search-oldest-first)
        (target-thread (notmuch-search-find-thread-id 'bare))