<div>Oops, yes.  I'm not sure why you had to initialize qparser_desc (are you sure it doesn't compile if you omit that?), but a change in the later patch 5/8 requires the extra argument to _notmuch_qparser_add_db_prefix.  I've got another patch with tests for patch 5/8 that adds and tests the argument that I'll send out shortly (along with tests for the remaining patches).</div>
<div><br></div><div>Glad to see you're taking advantage of the query parser!</div><br><div class="gmail_quote">On Sat, Jan 22, 2011 at 11:47 AM, Michal Sojka <span dir="ltr"><<a href="mailto:sojkam1@fel.cvut.cz">sojkam1@fel.cvut.cz</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div class="im">On Fri, 21 Jan 2011, Austin Clements wrote:<br>
> Since wildcard queries require a database, qparser-test can now open a<br>
> database.<br>
<br>
</div>Hi Austin,<br>
<br>
I had to apply the following changes in order to be able to compile the<br>
tests (make test).<br>
<br>
I'm going to test the parser in my daily use, but so far it looks really<br>
nice. I especially enjoy the before and after searches. Thanks.<br>
<br>
-Michal<br>
<div class="im"><br>
diff --git a/test/qparser-test.cc b/test/qparser-test.cc<br>
</div>index 18318aa..5be6220 100644<br>
<div class="im">--- a/test/qparser-test.cc<br>
+++ b/test/qparser-test.cc<br>
</div>@@ -61,7 +61,7 @@ test_one (void *ctx, const char *query_str)<br>
     void *local = talloc_new (ctx);<br>
     Xapian::Query q;<br>
     _notmuch_token_t *toks, *root;<br>
-    char *error, *qparser_desc, *xqparser_desc;<br>
+    char *error, *qparser_desc = NULL, *xqparser_desc;<br>
<br>
     toks = _notmuch_qparser_lex (local, qparser, query_str);<br>
     printf("[lex]    %s\n", _notmuch_token_show_list (local, toks));<br>
@@ -100,9 +100,9 @@ static _notmuch_qparser_t *<br>
 create_qparser (void *ctx)<br>
 {<br>
<div class="im">     _notmuch_qparser_t *qparser = _notmuch_qparser_create (ctx, notmuch);<br>
-    _notmuch_qparser_add_db_prefix (qparser, "prob", "P", FALSE);<br>
-    _notmuch_qparser_add_db_prefix (qparser, "lit", "L", TRUE);<br>
-    _notmuch_qparser_add_db_prefix (qparser, "tag", "K", TRUE);<br>
</div>+    _notmuch_qparser_add_db_prefix (qparser, "prob", "P", FALSE, FALSE);<br>
+    _notmuch_qparser_add_db_prefix (qparser, "lit", "L", TRUE, FALSE);<br>
+    _notmuch_qparser_add_db_prefix (qparser, "tag", "K", TRUE, FALSE);<br>
     return qparser;<br>
<div><div></div><div class="h5"> }<br>
<br>
_______________________________________________<br>
notmuch mailing list<br>
<a href="mailto:notmuch@notmuchmail.org">notmuch@notmuchmail.org</a><br>
<a href="http://notmuchmail.org/mailman/listinfo/notmuch" target="_blank">http://notmuchmail.org/mailman/listinfo/notmuch</a><br>
</div></div></blockquote></div><br>