emacs: jump: sort-order bugfix

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.
This commit is contained in:
Mark Walters 2014-09-02 19:56:04 +01:00 committed by David Bremner
parent 3d39d346d7
commit 5c4f6ed99b

View 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)))))