Mark Walters <markwalters1009@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.