e6/0b2db691dc360fbd21a90e3c327eccf26e4115

   1 Return-Path: <novalazy@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 2035F431FAF\r
   6         for <notmuch@notmuchmail.org>; Sat, 20 Oct 2012 19:04:36 -0700 (PDT)\r
   7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: -0.799\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.799 tagged_above=-999 required=5\r
  12         tests=[DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1,\r
  13         FREEMAIL_FROM=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 5slWtrzZ7RPR for <notmuch@notmuchmail.org>;\r
  17         Sat, 20 Oct 2012 19:04:35 -0700 (PDT)\r
  18 Received: from mail-pb0-f53.google.com (mail-pb0-f53.google.com\r
  19         [209.85.160.53]) (using TLSv1 with cipher RC4-SHA (128/128 bits))\r
  20         (No client certificate requested)\r
  21         by olra.theworths.org (Postfix) with ESMTPS id 81A0A431FAE\r
  22         for <notmuch@notmuchmail.org>; Sat, 20 Oct 2012 19:04:35 -0700 (PDT)\r
  23 Received: by mail-pb0-f53.google.com with SMTP id wz12so1476380pbc.26\r
  24         for <notmuch@notmuchmail.org>; Sat, 20 Oct 2012 19:04:34 -0700 (PDT)\r
  25 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113;\r
  26         h=date:message-id:from:to:subject:in-reply-to:references:mime-version\r
  27         :content-type:content-disposition:content-transfer-encoding;\r
  28         bh=IbqX1BQP85/4YfMRbkEpnUEjcnCmgMZMJ2JVcHyVjBY=;\r
  29         b=yOuJLiYLMNz6PXfdgRCqvkUOE1LJ1UjVuml4y47RS0wMHIg+xj+m4x99Ui15L04ehY\r
  30         GTOe55yYbLkqkYLBbztr1DNj6FOI6XDoEMF8rlpqmZ9VN9qa97xZYg1I5J5Lo9lLCcSx\r
  31         r2TIkGmjLvEkqURj5vTX7dl07snEpBTjmMUYJjx8cXkcEzR0+g+j6QWqeu72UAt3eEjo\r
  32         mjDu9jSYEgmQtJs0EMChZ31rprvWyFZBaZQsun5n9UKzrZI0bJwXiVtGomgTGssCau0C\r
  33         2qvTEp2sSHF9lX3AE9AvyiCGTR1CZ4jRBeNzVx3nLTT2c9OWoXiO4pXipau/OoQfD4ei\r
  34         3btg==\r
  35 Received: by 10.68.222.226 with SMTP id qp2mr9242477pbc.53.1350785074586;\r
  36         Sat, 20 Oct 2012 19:04:34 -0700 (PDT)\r
  37 Received: from localhost (215.42.233.220.static.exetel.com.au.\r
  38         [220.233.42.215])\r
  39         by mx.google.com with ESMTPS id ju7sm3603077pbb.60.2012.10.20.19.04.32\r
  40         (version=TLSv1/SSLv3 cipher=OTHER);\r
  41         Sat, 20 Oct 2012 19:04:33 -0700 (PDT)\r
  42 Date: Sun, 21 Oct 2012 13:04:27 +1100\r
  43 Message-ID: <20121021130427.GA4820@hili.localdomain>\r
  44 From: Peter Wang <novalazy@gmail.com>\r
  45 To: notmuch@notmuchmail.org\r
  46 Subject: Re: [PATCH 7/8] lib: add NOTMUCH_EXCLUDE_FLAG to notmuch_exclude_t\r
  47 In-Reply-To: <87txtr6o0c.fsf@betacantrips.com>\r
  48 References: <1340198947-29370-1-git-send-email-novalazy@gmail.com>\r
  49         <1340198947-29370-8-git-send-email-novalazy@gmail.com>\r
  50         <87txtr6o0c.fsf@betacantrips.com>\r
  51 MIME-Version: 1.0\r
  52 Content-Type: text/plain; charset=utf-8\r
  53 Content-Disposition: inline\r
  54 Content-Transfer-Encoding: 8bit\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, 21 Oct 2012 02:04:36 -0000\r
  68 \r
  69 On Fri, 19 Oct 2012 01:15:31 -0400, Ethan Glasser-Camp <ethan.glasser.camp@gmail.com> wrote:\r
  70 > Peter Wang <novalazy@gmail.com> writes:\r
  71 > \r
  72 > > Add NOTMUCH_EXCLUDE_FLAG to notmuch_exclude_t so that it can\r
  73 > > cover all four values of search --exclude in the cli.\r
  74 > \r
  75 > This series looks good to me. It's a nice clean up and a nice new\r
  76 > feature. Patches all apply.\r
  77 \r
  78 Thanks for the review.\r
  79 \r
  80 > However, I'm getting test failures like:\r
  81 > \r
  82 >  FAIL   Search, exclude "deleted" messages from message search --exclude=false\r
  83 >         --- excludes.3.expected 2012-10-19 04:45:06.900518377 +0000\r
  84 >         +++ excludes.3.output   2012-10-19 04:45:06.900518377 +0000\r
  85 >         @@ -1,2 +1,2 @@\r
  86 >         -id:msg-001@notmuch-test-suite\r
  87 >          id:msg-002@notmuch-test-suite\r
  88 >         +id:msg-001@notmuch-test-suite\r
  89 > \r
  90 >  FAIL   Search, don't exclude "deleted" messages when --exclude=flag specified\r
  91 >         --- excludes.7.expected 2012-10-19 04:45:07.004518378 +0000\r
  92 >         +++ excludes.7.output   2012-10-19 04:45:07.004518378 +0000\r
  93 >         @@ -1,2 +1,2 @@\r
  94 >         -thread:XXX   2001-01-05 [1/1] Notmuch Test Suite; Not deleted (inbox unread)\r
  95 >          thread:XXX   2001-01-05 [1/2] Notmuch Test Suite; Not deleted reply (deleted inbox unread)\r
  96 >         +thread:XXX   2001-01-05 [1/1] Notmuch Test Suite; Not deleted (inbox unread)\r
  97 > \r
  98 >  FAIL   Search, don't exclude "deleted" messages from search if not configured\r
  99 >         --- excludes.8.expected 2012-10-19 04:45:07.028518377 +0000\r
 100 >         +++ excludes.8.output   2012-10-19 04:45:07.028518377 +0000\r
 101 >         @@ -1,2 +1,2 @@\r
 102 >         -thread:XXX   2001-01-05 [1/1] Notmuch Test Suite; Not deleted (inbox unread)\r
 103 >          thread:XXX   2001-01-05 [2/2] Notmuch Test Suite; Deleted (deleted inbox unread)\r
 104 >         +thread:XXX   2001-01-05 [1/1] Notmuch Test Suite; Not deleted (inbox unread)\r
 105 > \r
 106 > In other words, threads and messages are coming up out of order. I'm not\r
 107 > sure of the right way to fix this. If you would like me to try sticking\r
 108 > "| sort" here and there in the tests I will do so. I'm not sure if the\r
 109 > test suite is guaranteed to scan messages in a certain order.\r
 110 \r
 111 Does it help if you add a "sleep 1" before the second generate_message\r
 112 call, i.e. on line 35?\r
 113 \r
 114 > > -         if (query->omit_excluded != NOTMUCH_EXCLUDE_FALSE)\r
 115 > > +         if (query->omit_excluded == NOTMUCH_EXCLUDE_TRUE ||\r
 116 > > +             query->omit_excluded == NOTMUCH_EXCLUDE_ALL)\r
 117 > > +         {\r
 118 > >               final_query = Xapian::Query (Xapian::Query::OP_AND_NOT,\r
 119 > >                                            final_query, exclude_query);\r
 120 > > -         else {\r
 121 > > +         } else {\r
 122 > \r
 123 > "House style" is to not put braces around one-line then-clauses. This is\r
 124 > the only place where you did that.\r
 125 \r
 126 I have to disagree.  The condition is wrapped over two lines.  The then\r
 127 part is wrapped over two lines.  The else part already has braces.\r
 128 All suggest braces around the then part.\r
 129 \r
 130 Peter\r