Xapian-quoting based batch-tagging.

Jani Nikula jani at nikula.org
Thu Jan 3 11:59:36 PST 2013


On Tue, 25 Dec 2012, david at tethera.net wrote:
> This is an alternative version of 
>
>      id:1356313183-9266-1-git-send-email-david at tethera.net
>
> batch tagging patches rebased on top of 
>
>      id:1356415076-5692-1-git-send-email-amdragon at mit.edu

I didn't go through this quite as thoroughly as the previous versions,
but the series LGTM. I like how this simplifies things. In the future,
I'd like us to support xapian quoting also in the tag changes, for
consistency.

BR,
Jani.



>
> This mainly consisted of removing 
>
>      [Patch v9 04/17] notmuch-tag: factor out double quoting routine
>      (superceded by one of Austin's patches)
>
>      [Patch v9 05/17] util/string-util: add a new string tokenized function
>      [Patch v9 06/17] unhex_and_quote: new function to quote hex-decoded queries
>      [Patch v9 07/17] notmuch-restore: move query handling for batch	
>      (uneeded if query is passed verbatim to xapian)
>
>      I also removed two tests, since they are about how we handle
>      quoting:
>
>      [Patch v9 13/17] test/tagging: add test for compound queries with batch tagging
>      [Patch v9 17/17] test/tagging: add test for handling of parenthesized	tag queries.
>
> A few small fixes were needed to the tests, and a fair amount of
> changes to the notmuch-tag man page.
>
> Diffstat (against Austin's series) is as follows
>
>  man/man1/notmuch-tag.1 |   99 ++++++++++++++++++++-
>  notmuch-tag.c          |  169 ++++++++++++++++++++----------------
>  tag-util.c             |   87 +++++++++++++++++--
>  tag-util.h             |   15 ++++
>  test/tagging           |  195 ++++++++++++++++++++++++++++++++++++++++++
>  5 files changed, 480 insertions(+), 85 deletions(-)
>
>
> _______________________________________________
> notmuch mailing list
> notmuch at notmuchmail.org
> http://notmuchmail.org/mailman/listinfo/notmuch


More information about the notmuch mailing list