From bed62eb8bee4aeca1fabfa5e302b515849f50b31 Mon Sep 17 00:00:00 2001 From: David Bremner Date: Tue, 6 Jul 2021 22:51:08 -0300 Subject: [PATCH] emacs: remove useless lexically bound variable A let binding without a value is just an obfuscated way of saying nil, especially if you are not going to mutate the variable. --- emacs/notmuch-tree.el | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/emacs/notmuch-tree.el b/emacs/notmuch-tree.el index b288c05d..eb7ea941 100644 --- a/emacs/notmuch-tree.el +++ b/emacs/notmuch-tree.el @@ -1001,10 +1001,9 @@ message together with all its descendents." (defun notmuch-tree-insert-forest-thread (forest-thread) "Insert a single complete thread." - (let (tree-status) - ;; Reset at the start of each main thread. - (setq notmuch-tree-previous-subject nil) - (notmuch-tree-insert-thread forest-thread 0 tree-status))) + ;; Reset at the start of each main thread. + (setq notmuch-tree-previous-subject nil) + (notmuch-tree-insert-thread forest-thread 0 nil)) (defun notmuch-tree-insert-forest (forest) "Insert a forest of threads. -- 2.43.0