emacs: jump: sort-order bugfix
authorMark Walters <markwalters1009@gmail.com>
Tue, 2 Sep 2014 18:56:04 +0000 (19:56 +0100)
committerDavid Bremner <david@tethera.net>
Sun, 7 Sep 2014 18:02:16 +0000 (20:02 +0200)
default-value needs its argument to be quoted.

Slightly strangely default-value of 't or nil is 't or nil
respectively so the code

(default-value notmuch-search-oldest-first)

just gives the current value of notmuch-search-oldest-first rather
than intended default-value of this variable.

The symptom is that if you are in a search buffer and use notmuch jump
to run a saved search which does not have an explicitly set sort order
then the sort order of the saved-search is inherited from the current
search buffer rather than being the default search order.

Thanks to Jani for finding the bug.

emacs/notmuch-jump.el

index 5eb0949be0c3757f2e26c84ad6bb3c42b978062b..0193f8cd7064436efb73f13da4ea798668cbfef0 100644 (file)
@@ -51,7 +51,7 @@ fast way to jump to a saved search from anywhere in Notmuch."
                 (case (plist-get saved-search :sort-order)
                   (newest-first nil)
                   (oldest-first t)
-                  (otherwise (default-value notmuch-search-oldest-first)))))
+                  (otherwise (default-value 'notmuch-search-oldest-first)))))
            (push (list key name
                        `(lambda () (notmuch-search ',query ',oldest-first)))
                  action-map)))))