From: Tomi Ollila <tomi.ollila@iki.fi>
To: Mark Walters <markwalters1009@gmail.com>, notmuch@notmuchmail.org
Subject: Re: [PATCH] emacs: search archive tweak
In-Reply-To: <1402058914-16070-1-git-send-email-markwalters1009@gmail.com>
References: <87bnv92521.fsf@inf-8660.int-evry.fr>
	<1402058914-16070-1-git-send-email-markwalters1009@gmail.com>
User-Agent: Notmuch/0.18+28~gcecaba1 (http://notmuchmail.org) Emacs/24.3.1
	(x86_64-unknown-linux-gnu)
Date: Sat, 07 Jun 2014 16:38:29 +0300
Message-ID: <m2d2ekn8kq.fsf@guru.guru-group.fi>
MIME-Version: 1.0
Content-Type: text/plain
Precedence: list

On Fri, Jun 06 2014, Mark Walters <markwalters1009@gmail.com> wrote:

> notmuch-search-archive-thread moves to the next line after tagging. In
> the normal case this makes sense, but if the region is active, it tags
> the whole region and then it doesn't really. Thus only move to the
> next line if region is not active.
> ---

Looks good, although I have no idea how to test this ;)

Tomi

>
> In response to the request in the parent message.
>
> Best wishes
>
> Mark
>
>  emacs/notmuch.el |    3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/emacs/notmuch.el b/emacs/notmuch.el
> index f6bf9c8..2a9876f 100644
> --- a/emacs/notmuch.el
> +++ b/emacs/notmuch.el
> @@ -580,7 +580,8 @@ (defun notmuch-search-archive-thread (&optional unarchive beg end)
>    (when notmuch-archive-tags
>      (notmuch-search-tag
>       (notmuch-tag-change-list notmuch-archive-tags unarchive) beg end))
> -  (notmuch-search-next-thread))
> +  (when (eq beg end)
> +    (notmuch-search-next-thread)))
>  
>  (defun notmuch-search-update-result (result &optional pos)
>    "Replace the result object of the thread at POS (or point) by
> -- 
> 1.7.10.4
>
> _______________________________________________
> notmuch mailing list
> notmuch@notmuchmail.org
> http://notmuchmail.org/mailman/listinfo/notmuch