From: Mark Walters <markwalters1009@gmail.com>
To: notmuch@notmuchmail.org
Subject: Re: [PATCH 2/5] emacs: make notmuch-show return its buffer
In-Reply-To: <1343164911-31589-3-git-send-email-markwalters1009@gmail.com>
References: <1343164911-31589-1-git-send-email-markwalters1009@gmail.com>
	<1343164911-31589-3-git-send-email-markwalters1009@gmail.com>
User-Agent: Notmuch/0.13.2+93~gf33b188 (http://notmuchmail.org) Emacs/23.4.1
	(x86_64-pc-linux-gnu)
Date: Sat, 28 Jul 2012 18:01:49 +0100
Message-ID: <87ipd7vmyq.fsf@qmul.ac.uk>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Precedence: list


Hi

I have a version 2 of the rest of this series. However, it obviously
conflicts with the two patches I submitted today splitting out the
async-json parser and the clean-author function. Hence I will see how
they are received before submitting version 2. 

I will mark Patches 3-5 obsolete (and have already marked 1 obsolete as
Tomi had a much better way of doing it). However Patch 2/5 will be needed in
any case so review/approval for this one-line patch would be great!

Many thanks

Mark


On Tue, 24 Jul 2012, Mark Walters <markwalters1009@gmail.com> wrote:
> notmuch-pick uses the returned buffer to try and make sure it does not
> close the wrong buffer.
> ---
>  emacs/notmuch-show.el |    3 ++-
>  1 files changed, 2 insertions(+), 1 deletions(-)
>
> diff --git a/emacs/notmuch-show.el b/emacs/notmuch-show.el
> index 6335d45..1cabd17 100644
> --- a/emacs/notmuch-show.el
> +++ b/emacs/notmuch-show.el
> @@ -1031,7 +1031,8 @@ function is used."
>  	  notmuch-show-parent-buffer parent-buffer
>  	  notmuch-show-query-context query-context)
>      (notmuch-show-build-buffer)
> -    (notmuch-show-goto-first-wanted-message)))
> +    (notmuch-show-goto-first-wanted-message)
> +    (current-buffer)))
>  
>  (defun notmuch-show-build-buffer ()
>    (let ((inhibit-read-only t))
> -- 
> 1.7.9.1