[PATCH 2/3] emacs: use a single history for all searches

2012-01-18 Thread Dmitry Kurochkin
On Wed, 18 Jan 2012 08:25:22 +, David Edmondson wrote: > On Tue, 17 Jan 2012 23:34:09 +0400, Dmitry Kurochkin gmail.com> wrote: > > There are two ways to do search in Emacs UI: search widget in > > notmuch-hello buffer and `notmuch-search' function bound to "s". > > Before the change, these

[PATCH 2/3] emacs: use a single history for all searches

2012-01-18 Thread David Edmondson
On Wed, 18 Jan 2012 21:57:27 +0400, Dmitry Kurochkin wrote: > > The test output updates included here should be with the previous patch, > > shouldn't they? > > Indeed. Would send v2. That's not necessary for me. > > Should user-typed queries be trimmed? > > IMO that would not hurt. But we

[PATCH 2/3] emacs: use a single history for all searches

2012-01-18 Thread David Edmondson
On Tue, 17 Jan 2012 23:34:09 +0400, Dmitry Kurochkin wrote: > There are two ways to do search in Emacs UI: search widget in > notmuch-hello buffer and `notmuch-search' function bound to "s". > Before the change, these search mechanisms used different history > lists. The patch makes

Re: [PATCH 2/3] emacs: use a single history for all searches

2012-01-18 Thread David Edmondson
On Tue, 17 Jan 2012 23:34:09 +0400, Dmitry Kurochkin dmitry.kuroch...@gmail.com wrote: There are two ways to do search in Emacs UI: search widget in notmuch-hello buffer and `notmuch-search' function bound to s. Before the change, these search mechanisms used different history lists. The

Re: [PATCH 2/3] emacs: use a single history for all searches

2012-01-18 Thread Dmitry Kurochkin
On Wed, 18 Jan 2012 08:25:22 +, David Edmondson d...@dme.org wrote: On Tue, 17 Jan 2012 23:34:09 +0400, Dmitry Kurochkin dmitry.kuroch...@gmail.com wrote: There are two ways to do search in Emacs UI: search widget in notmuch-hello buffer and `notmuch-search' function bound to s.

Re: [PATCH 2/3] emacs: use a single history for all searches

2012-01-18 Thread David Edmondson
On Wed, 18 Jan 2012 21:57:27 +0400, Dmitry Kurochkin dmitry.kuroch...@gmail.com wrote: The test output updates included here should be with the previous patch, shouldn't they? Indeed. Would send v2. That's not necessary for me. Should user-typed queries be trimmed? IMO that would

[PATCH 2/3] emacs: use a single history for all searches

2012-01-17 Thread Dmitry Kurochkin
There are two ways to do search in Emacs UI: search widget in notmuch-hello buffer and `notmuch-search' function bound to "s". Before the change, these search mechanisms used different history lists. The patch makes notmuch-hello search use the same history list as `notmuch-search' function. ---

[PATCH 2/3] emacs: use a single history for all searches

2012-01-17 Thread Dmitry Kurochkin
There are two ways to do search in Emacs UI: search widget in notmuch-hello buffer and `notmuch-search' function bound to s. Before the change, these search mechanisms used different history lists. The patch makes notmuch-hello search use the same history list as `notmuch-search' function. ---