Mark Walters <markwalters1009 at gmail.com> writes: > + OPEN-TARGET: If TRUE open the target message in the message pane. > + OLDEST-FIRST: If TRUE display threads sorted oldest first" > + (interactive > + (list > + ;; Prompt for a query > + nil > + ;; use default search order > + nil nil nil nil (default-value 'notmuch-search-oldest-first)))
This is probably just me being ignorant, but I had to check the docs for this usage of interactive. If you think it's tricky also, might be worth a brief comment. Otherwise the series looks ok to me.