From b8fe20f339677ae54bbc1bc606bd08ef551a2974 Mon Sep 17 00:00:00 2001 From: David Bremner Date: Sat, 22 Jul 2023 09:15:56 -0300 Subject: [PATCH] emacs/hello: refresh hello directly According to the now deleted commentary, the hack of using run-at-time was needed for Emacs 24. It seems to be no longer needed for Emacs 28.2, and removing it makes further changes to the code simpler. --- emacs/notmuch-hello.el | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/emacs/notmuch-hello.el b/emacs/notmuch-hello.el index b3228d43..15ba7c78 100644 --- a/emacs/notmuch-hello.el +++ b/emacs/notmuch-hello.el @@ -710,11 +710,7 @@ with `notmuch-hello-query-counts'." ;; configuration change, and this is not a new window) (setq do-refresh t))))) (when (and do-refresh notmuch-hello-auto-refresh) - ;; Refresh hello as soon as we get back to redisplay. On Emacs - ;; 24, we can't do it right here because something in this - ;; hook's call stack overrides hello's point placement. - ;; FIXME And on Emacs releases that we still support? - (run-at-time nil nil #'notmuch-hello t)) + (notmuch-hello t)) (unless hello-buf ;; Clean up hook (remove-hook 'window-configuration-change-hook