Re: [PATCH v2] emacs: notmuch search bugfix

Subject: Re: [PATCH v2] emacs: notmuch search bugfix

Date: Tue, 07 Aug 2012 14:26:37 -0400

To: Mark Walters

Cc: notmuch@notmuchmail.org

From: Austin Clements


On Tue, 07 Aug 2012, Mark Walters <markwalters1009@gmail.com> wrote:
> The recent change to use json for notmuch-search.el introduced a bug
> in the code for keeping position on refresh. The problem is a
> comparison between (plist-get result :thread) and a thread-id returned
> by notmuch-search-find-thread-id: the latter is prefixed with
> "thread:"
>
> We fix this by adding an option to notmuch-search-find-thread-id to
> return the bare thread-id. It appears that notmuch-search-refresh-view
> is the only caller of notmuch-search that supplies a thread-id so this
> change should be safe (but could theoretically break users .emacs
> functions).
> ---
>
> I think this implements all of Austin's suggestions, and it adds a short
> remark to NEWS (since in unlikely cases it could break users' functions)
> and updates the docstring for notmuch-search. 

LGTM.

Thread: