From: Mark Walters <markwalters1009@gmail.com>
To: david@tethera.net, notmuch@notmuchmail.org, 
Subject: Re: Xapian-quoting based batch-tagging.
In-Reply-To: <1356464567-21779-1-git-send-email-david@tethera.net>
References: <1356464567-21779-1-git-send-email-david@tethera.net>
User-Agent: Notmuch/0.14+236~g1d0044f (http://notmuchmail.org) Emacs/23.4.1
	(x86_64-pc-linux-gnu)
Date: Wed, 26 Dec 2012 08:52:35 +0000
Message-ID: <87ehid5h64.fsf@qmul.ac.uk>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Precedence: list


On Tue, 25 Dec 2012, david@tethera.net wrote:
> This is an alternative version of 
>
>      id:1356313183-9266-1-git-send-email-david@tethera.net
>
> batch tagging patches rebased on top of 
>
>      id:1356415076-5692-1-git-send-email-amdragon@mit.edu
>
> 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.

Hi

I am unclear about how this is going to deal with queries containing
newlines. For dump/restore I think this is not a problem (as Austin and
others have said), but for batch tagging I think it could be; for
example the query could be for a tag containing a newline.

Best wishes

Mark


>
> 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@notmuchmail.org
> http://notmuchmail.org/mailman/listinfo/notmuch