Re: [PATCH v2] emacs: add support for stashing the thread id in show view

Subject: Re: [PATCH v2] emacs: add support for stashing the thread id in show view

Date: Fri, 07 Dec 2012 07:53:18 +0000

To: Jani Nikula, notmuch@notmuchmail.org

Cc:

From: Mark Walters


Jani Nikula <jani@nikula.org> writes:

> Add a prefix argument to notmuch-show-stash-message-id to stash thread
> id instead of message id.


LGTM +1

Mark


> ---
>  emacs/notmuch-show.el |   14 ++++++++++----
>  1 file changed, 10 insertions(+), 4 deletions(-)
>
> diff --git a/emacs/notmuch-show.el b/emacs/notmuch-show.el
> index 4d6c014..658a2be 100644
> --- a/emacs/notmuch-show.el
> +++ b/emacs/notmuch-show.el
> @@ -1901,10 +1901,16 @@ thread from search."
>    (interactive)
>    (notmuch-common-do-stash (notmuch-show-get-from)))
>  
> -(defun notmuch-show-stash-message-id ()
> -  "Copy id: query matching the current message to kill-ring."
> -  (interactive)
> -  (notmuch-common-do-stash (notmuch-show-get-message-id)))
> +(defun notmuch-show-stash-message-id (&optional stash-thread-id)
> +  "Copy id: query matching the current message to kill-ring.
> +
> +If invoked with a prefix argument (or STASH-THREAD-ID is
> +non-nil), copy thread: query matching the current thread to
> +kill-ring."
> +  (interactive "P")
> +  (if stash-thread-id
> +      (notmuch-common-do-stash notmuch-show-thread-id)
> +    (notmuch-common-do-stash (notmuch-show-get-message-id))))
>  
>  (defun notmuch-show-stash-message-id-stripped ()
>    "Copy message ID of current message (sans `id:' prefix) to kill-ring."
> -- 
> 1.7.10.4

Thread: