[PATCH 3.5/8] Query parser tests for wildcard queries.

2011-01-24 Thread Michal Sojka
On Sun, 23 Jan 2011, Austin Clements wrote: > Oops, yes. I'm not sure why you had to initialize qparser_desc (are you > sure it doesn't compile if you omit that?), This was another problem - I got a warning that this variable might be uninitialized. -Michal

[PATCH 3.5/8] Query parser tests for wildcard queries.

2011-01-23 Thread Austin Clements
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

[PATCH 3.5/8] Query parser tests for wildcard queries.

2011-01-22 Thread Michal Sojka
On Fri, 21 Jan 2011, Austin Clements wrote: > Since wildcard queries require a database, qparser-test can now open a > database. Hi Austin, I had to apply the following changes in order to be able to compile the tests (make test). I'm going to test the parser in my daily use, but so far it

Re: [PATCH 3.5/8] Query parser tests for wildcard queries.

2011-01-22 Thread Michal Sojka
On Fri, 21 Jan 2011, Austin Clements wrote: Since wildcard queries require a database, qparser-test can now open a database. Hi Austin, I had to apply the following changes in order to be able to compile the tests (make test). I'm going to test the parser in my daily use, but so far it looks

[PATCH 3.5/8] Query parser tests for wildcard queries.

2011-01-21 Thread Austin Clements
Since wildcard queries require a database, qparser-test can now open a database. --- This is intended to be applied after patch 3/5 in this series, id:1295165458-9573-4-git-send-email-amdragon at mit.edu test/qparser |7 +++ test/qparser-test.cc