Re: [Patch v7 06/14] notmuch-restore: move query handling for batch restore to parser

Subject: Re: [Patch v7 06/14] notmuch-restore: move query handling for batch restore to parser

Date: Sat, 15 Dec 2012 14:18:23 -0400

To: Mark Walters, notmuch@notmuchmail.org

Cc:

From: David Bremner


Mark Walters <markwalters1009@gmail.com> writes:

>> +	if (hex_decode_inplace (tok) != HEX_SUCCESS) {
>> +	    ret = line_error (TAG_PARSE_INVALID, line_for_error,
>> +			      "hex decoding of query %s failed", tok);
>> +	    goto DONE;
>> +	}
>> +	/* skip 'id:' */
>> +	*query_string = tok + 3;
>
> This looks like it doesn't double_quote the query_string in this (the
> TAG_FLAG_ID_ONLY) case. Is that deliberate?

Yes, that's what I meant by 

,----
| We also need this to avoid the query quoting for more
| general queries (to be written) that will mess up raw message-ids.
`----

perhaps it deserves a comment in the code.

Thread: