v2 regexp search for mid/folder/path

2017-03-29 Thread David Bremner
This is mainly a rebase against current master, but also required an update to the (just pushed) empty query string handling Apparently there's at least 4 ways to test a C++ string for being empty, so here I try a different way of writing it. Interdiff follows: diff --git a/lib/regexp-fields.cc

[PATCH 1/2] lib: Add regexp searching for mid: prefix

2017-03-29 Thread David Bremner
The bulk of the change is passing in the field options to the regexp field processor, so that we can properly handle the fallback (non-regexp case). --- lib/database.cc | 6 -- lib/regexp-fields.cc | 36 +--- lib/regexp-fields.h | 4 +++-

[PATCH 2/2] lib: Add regexp expansion for for tags and paths

2017-03-29 Thread David Bremner
From a ui perspective this looks similar to what was already provided for from, subject, and mid, but the implimentation is quite different. It uses the database's list of terms to construct a term based query equivalent to the passed regular expression. --- lib/database.cc | 12

Re: revised foo:"" handling

2017-03-29 Thread David Bremner
Tomi Ollila writes: > On Sat, Mar 25 2017, David Bremner wrote: > >> This obsoletes the first two patches of >> >> id:20170318030303.17344-1-da...@tethera.net >> >> I think this is a more meaningful interpretation than matching all messages. > >

Re: [PATCH] NEWS: initial NEWS changes for 0.24.1

2017-03-29 Thread David Bremner
David Bremner writes: > I expect these to be updated as a few more patches are added to the release. I pushed an updated version. ___ notmuch mailing list notmuch@notmuchmail.org https://notmuchmail.org/mailman/listinfo/notmuch

Re: revised foo:"" handling

2017-03-29 Thread Tomi Ollila
On Sat, Mar 25 2017, David Bremner wrote: > This obsoletes the first two patches of > > id:20170318030303.17344-1-da...@tethera.net > > I think this is a more meaningful interpretation than matching all messages. These changes look good (AFAIU). tests pass (debian

Re: "search --path=directory/" is lame(-ish)

2017-03-29 Thread David Bremner
David Edmondson writes: > Adding a terminal slash to a directory name when using --path causes the > search to fail. Removing the terminal slash produces results. > > Given that many shells will add the terminal slash during completion, > this is lame(-ish). This would be