eb/e0301da1356e6fbcc96e3504f7946c827031a4

   1 Return-Path: <amdragon@gmail.com>\r
   2 X-Original-To: notmuch@notmuchmail.org\r
   3 Delivered-To: notmuch@notmuchmail.org\r
   4 Received: from localhost (localhost [127.0.0.1])\r
   5         by olra.theworths.org (Postfix) with ESMTP id 275C0431FD0\r
   6         for <notmuch@notmuchmail.org>; Sun, 23 Jan 2011 14:02:47 -0800 (PST)\r
   7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: -0.698\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.698 tagged_above=-999 required=5\r
  12         tests=[DKIM_SIGNED=0.1, DKIM_VALID=-0.1, FREEMAIL_FROM=0.001,\r
  13         HTML_MESSAGE=0.001, RCVD_IN_DNSWL_LOW=-0.7] autolearn=disabled\r
  14 Received: from olra.theworths.org ([127.0.0.1])\r
  15         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  16         with ESMTP id 6chJZHClJqg7 for <notmuch@notmuchmail.org>;\r
  17         Sun, 23 Jan 2011 14:02:46 -0800 (PST)\r
  18 Received: from mail-qw0-f53.google.com (mail-qw0-f53.google.com\r
  19         [209.85.216.53]) (using TLSv1 with cipher RC4-MD5 (128/128 bits))\r
  20         (No client certificate requested)\r
  21         by olra.theworths.org (Postfix) with ESMTPS id 43369431FB6\r
  22         for <notmuch@notmuchmail.org>; Sun, 23 Jan 2011 14:02:46 -0800 (PST)\r
  23 Received: by qwe5 with SMTP id 5so3689310qwe.26\r
  24         for <notmuch@notmuchmail.org>; Sun, 23 Jan 2011 14:02:45 -0800 (PST)\r
  25 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma;\r
  26         h=domainkey-signature:mime-version:sender:in-reply-to:references:date\r
  27         :x-google-sender-auth:message-id:subject:from:to:cc:content-type;\r
  28         bh=UR8QYuUG8D+qs+EJ/3uE8gkD2d5qLt/AAXqTUmL3uRI=;\r
  29         b=kDtyzFXLkz7AvJ1fz+VCeL8vgkCiUV1xItP1wiEAmZ+pSYeLRiQtsvBs+DBts/CMtv\r
  30         FlrSk+IK/b07SuBFE9r23smQsloCdQYIrsee1543J69j9h3Hfy5OAV/xQoBy322zrP2T\r
  31         OEuyoUxTFzD/6OsSEQCRCxFqYagSDEKITjH68=\r
  32 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma;\r
  33         h=mime-version:sender:in-reply-to:references:date\r
  34         :x-google-sender-auth:message-id:subject:from:to:cc:content-type;\r
  35         b=FjH8y1imG4XZeCtYNWzv3y/3lotthYVYqXN03yIPHi8Cah8Fnftiyy04XP9UOMMZSN\r
  36         pxY8FpzzHCkK6+GaqBD9nB6BvDP7qidt90U2W4qpI0oLotSB0GTc306Jevim+ORszf8o\r
  37         j0R1OIL9VTkHvM3b/lWeGXEFBmf60zoWqYXMc=\r
  38 MIME-Version: 1.0\r
  39 Received: by 10.229.217.133 with SMTP id hm5mr3137718qcb.40.1295820163376;\r
  40         Sun, 23 Jan 2011 14:02:43 -0800 (PST)\r
  41 Sender: amdragon@gmail.com\r
  42 Received: by 10.229.97.143 with HTTP; Sun, 23 Jan 2011 14:02:43 -0800 (PST)\r
  43 In-Reply-To: <87pqrosx6e.fsf@steelpick.2x.cz>\r
  44 References: <1295165458-9573-1-git-send-email-amdragon@mit.edu>\r
  45         <1295165458-9573-4-git-send-email-amdragon@mit.edu>\r
  46         <20110121064003.GK13226@mit.edu> <87pqrosx6e.fsf@steelpick.2x.cz>\r
  47 Date: Sun, 23 Jan 2011 17:02:43 -0500\r
  48 X-Google-Sender-Auth: JyOYyyFSPYKP2UhXVPQ_LysGUEY\r
  49 Message-ID: <AANLkTi==CxAUVvtO6nbFo_XPWmGo4u1MLH8LcQuquyAE@mail.gmail.com>\r
  50 Subject: Re: [PATCH 3.5/8] Query parser tests for wildcard queries.\r
  51 From: Austin Clements <amdragon@mit.edu>\r
  52 To: Michal Sojka <sojkam1@fel.cvut.cz>\r
  53 Content-Type: multipart/alternative; boundary=0016361e81a287e719049a8aa496\r
  54 Cc: notmuch@notmuchmail.org\r
  55 X-BeenThere: notmuch@notmuchmail.org\r
  56 X-Mailman-Version: 2.1.13\r
  57 Precedence: list\r
  58 List-Id: "Use and development of the notmuch mail system."\r
  59         <notmuch.notmuchmail.org>\r
  60 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  61         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  62 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  63 List-Post: <mailto:notmuch@notmuchmail.org>\r
  64 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  65 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  66         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  67 X-List-Received-Date: Sun, 23 Jan 2011 22:02:47 -0000\r
  68 \r
  69 --0016361e81a287e719049a8aa496\r
  70 Content-Type: text/plain; charset=ISO-8859-1\r
  71 \r
  72 Oops, yes.  I'm not sure why you had to initialize qparser_desc (are you\r
  73 sure it doesn't compile if you omit that?), but a change in the later patch\r
  74 5/8 requires the extra argument to _notmuch_qparser_add_db_prefix.  I've got\r
  75 another patch with tests for patch 5/8 that adds and tests the argument that\r
  76 I'll send out shortly (along with tests for the remaining patches).\r
  77 \r
  78 Glad to see you're taking advantage of the query parser!\r
  79 \r
  80 On Sat, Jan 22, 2011 at 11:47 AM, Michal Sojka <sojkam1@fel.cvut.cz> wrote:\r
  81 \r
  82 > On Fri, 21 Jan 2011, Austin Clements wrote:\r
  83 > > Since wildcard queries require a database, qparser-test can now open a\r
  84 > > database.\r
  85 >\r
  86 > Hi Austin,\r
  87 >\r
  88 > I had to apply the following changes in order to be able to compile the\r
  89 > tests (make test).\r
  90 >\r
  91 > I'm going to test the parser in my daily use, but so far it looks really\r
  92 > nice. I especially enjoy the before and after searches. Thanks.\r
  93 >\r
  94 > -Michal\r
  95 >\r
  96 > diff --git a/test/qparser-test.cc b/test/qparser-test.cc\r
  97 > index 18318aa..5be6220 100644\r
  98 > --- a/test/qparser-test.cc\r
  99 > +++ b/test/qparser-test.cc\r
 100 > @@ -61,7 +61,7 @@ test_one (void *ctx, const char *query_str)\r
 101 >     void *local = talloc_new (ctx);\r
 102 >     Xapian::Query q;\r
 103 >     _notmuch_token_t *toks, *root;\r
 104 > -    char *error, *qparser_desc, *xqparser_desc;\r
 105 > +    char *error, *qparser_desc = NULL, *xqparser_desc;\r
 106 >\r
 107 >     toks = _notmuch_qparser_lex (local, qparser, query_str);\r
 108 >     printf("[lex]    %s\n", _notmuch_token_show_list (local, toks));\r
 109 > @@ -100,9 +100,9 @@ static _notmuch_qparser_t *\r
 110 >  create_qparser (void *ctx)\r
 111 >  {\r
 112 >      _notmuch_qparser_t *qparser = _notmuch_qparser_create (ctx, notmuch);\r
 113 > -    _notmuch_qparser_add_db_prefix (qparser, "prob", "P", FALSE);\r
 114 > -    _notmuch_qparser_add_db_prefix (qparser, "lit", "L", TRUE);\r
 115 > -    _notmuch_qparser_add_db_prefix (qparser, "tag", "K", TRUE);\r
 116 > +    _notmuch_qparser_add_db_prefix (qparser, "prob", "P", FALSE, FALSE);\r
 117 > +    _notmuch_qparser_add_db_prefix (qparser, "lit", "L", TRUE, FALSE);\r
 118 > +    _notmuch_qparser_add_db_prefix (qparser, "tag", "K", TRUE, FALSE);\r
 119 >     return qparser;\r
 120 >  }\r
 121 >\r
 122 > _______________________________________________\r
 123 > notmuch mailing list\r
 124 > notmuch@notmuchmail.org\r
 125 > http://notmuchmail.org/mailman/listinfo/notmuch\r
 126 >\r
 127 \r
 128 --0016361e81a287e719049a8aa496\r
 129 Content-Type: text/html; charset=ISO-8859-1\r
 130 Content-Transfer-Encoding: quoted-printable\r
 131 \r
 132 <div>Oops, yes. =A0I&#39;m not sure why you had to initialize qparser_desc =\r
 133 (are you sure it doesn&#39;t compile if you omit that?), but a change in th=\r
 134 e later patch 5/8 requires the extra argument to _notmuch_qparser_add_db_pr=\r
 135 efix. =A0I&#39;ve got another patch with tests for patch 5/8 that adds and =\r
 136 tests the argument that I&#39;ll send out shortly (along with tests for the=\r
 137  remaining patches).</div>\r
 138 <div><br></div><div>Glad to see you&#39;re taking advantage of the query pa=\r
 139 rser!</div><br><div class=3D"gmail_quote">On Sat, Jan 22, 2011 at 11:47 AM,=\r
 140  Michal Sojka <span dir=3D"ltr">&lt;<a href=3D"mailto:sojkam1@fel.cvut.cz">=\r
 141 sojkam1@fel.cvut.cz</a>&gt;</span> wrote:<br>\r
 142 <blockquote class=3D"gmail_quote" style=3D"margin:0 0 0 .8ex;border-left:1p=\r
 143 x #ccc solid;padding-left:1ex;"><div class=3D"im">On Fri, 21 Jan 2011, Aust=\r
 144 in Clements wrote:<br>\r
 145 &gt; Since wildcard queries require a database, qparser-test can now open a=\r
 146 <br>\r
 147 &gt; database.<br>\r
 148 <br>\r
 149 </div>Hi Austin,<br>\r
 150 <br>\r
 151 I had to apply the following changes in order to be able to compile the<br>\r
 152 tests (make test).<br>\r
 153 <br>\r
 154 I&#39;m going to test the parser in my daily use, but so far it looks reall=\r
 155 y<br>\r
 156 nice. I especially enjoy the before and after searches. Thanks.<br>\r
 157 <br>\r
 158 -Michal<br>\r
 159 <div class=3D"im"><br>\r
 160 diff --git a/test/qparser-test.cc b/test/qparser-test.cc<br>\r
 161 </div>index 18318aa..5be6220 100644<br>\r
 162 <div class=3D"im">--- a/test/qparser-test.cc<br>\r
 163 +++ b/test/qparser-test.cc<br>\r
 164 </div>@@ -61,7 +61,7 @@ test_one (void *ctx, const char *query_str)<br>\r
 165  =A0 =A0 void *local =3D talloc_new (ctx);<br>\r
 166  =A0 =A0 Xapian::Query q;<br>\r
 167  =A0 =A0 _notmuch_token_t *toks, *root;<br>\r
 168 - =A0 =A0char *error, *qparser_desc, *xqparser_desc;<br>\r
 169 + =A0 =A0char *error, *qparser_desc =3D NULL, *xqparser_desc;<br>\r
 170 <br>\r
 171  =A0 =A0 toks =3D _notmuch_qparser_lex (local, qparser, query_str);<br>\r
 172  =A0 =A0 printf(&quot;[lex] =A0 =A0%s\n&quot;, _notmuch_token_show_list (lo=\r
 173 cal, toks));<br>\r
 174 @@ -100,9 +100,9 @@ static _notmuch_qparser_t *<br>\r
 175 =A0create_qparser (void *ctx)<br>\r
 176 =A0{<br>\r
 177 <div class=3D"im"> =A0 =A0 _notmuch_qparser_t *qparser =3D _notmuch_qparser=\r
 178 _create (ctx, notmuch);<br>\r
 179 - =A0 =A0_notmuch_qparser_add_db_prefix (qparser, &quot;prob&quot;, &quot;P=\r
 180 &quot;, FALSE);<br>\r
 181 - =A0 =A0_notmuch_qparser_add_db_prefix (qparser, &quot;lit&quot;, &quot;L&=\r
 182 quot;, TRUE);<br>\r
 183 - =A0 =A0_notmuch_qparser_add_db_prefix (qparser, &quot;tag&quot;, &quot;K&=\r
 184 quot;, TRUE);<br>\r
 185 </div>+ =A0 =A0_notmuch_qparser_add_db_prefix (qparser, &quot;prob&quot;, &=\r
 186 quot;P&quot;, FALSE, FALSE);<br>\r
 187 + =A0 =A0_notmuch_qparser_add_db_prefix (qparser, &quot;lit&quot;, &quot;L&=\r
 188 quot;, TRUE, FALSE);<br>\r
 189 + =A0 =A0_notmuch_qparser_add_db_prefix (qparser, &quot;tag&quot;, &quot;K&=\r
 190 quot;, TRUE, FALSE);<br>\r
 191  =A0 =A0 return qparser;<br>\r
 192 <div><div></div><div class=3D"h5">=A0}<br>\r
 193 <br>\r
 194 _______________________________________________<br>\r
 195 notmuch mailing list<br>\r
 196 <a href=3D"mailto:notmuch@notmuchmail.org">notmuch@notmuchmail.org</a><br>\r
 197 <a href=3D"http://notmuchmail.org/mailman/listinfo/notmuch" target=3D"_blan=\r
 198 k">http://notmuchmail.org/mailman/listinfo/notmuch</a><br>\r
 199 </div></div></blockquote></div><br>\r
 200 \r
 201 --0016361e81a287e719049a8aa496--\r