Re: [PATCH v2] emacs: wrap current search in parens when filtering

2015-09-06 Thread Tomi Ollila
On Sun, Sep 06 2015, David Bremner wrote: > Some pretty fussy comments follow. Probably I could have fixed these in > the time it took to write this message ;). > > Uli Scholler writes: >> + (let ((grouped-query (notmuch-maybe-group-query-string query)) >>

Re: [PATCH v2] emacs: wrap current search in parens when filtering

2015-09-05 Thread David Bremner
Some pretty fussy comments follow. Probably I could have fixed these in the time it took to write this message ;). Uli Scholler writes: > + (let ((grouped-query (notmuch-maybe-group-query-string query)) > + (grouped-search-query (notmuch-maybe-group-query-string >

[PATCH v2] emacs: wrap current search in parens when filtering

2015-09-03 Thread Uli Scholler
Tomi Ollila writes: > But should this do the same "notmuch-search-disjunctive-regexp" check > which is done when building up `grouped-query' ? Here is an improved version of my patch: --- When filtering the current search further with notmuch-search-filter, wrap the current