9d/d39bad749bb3efc173c04fa3f582f40f3591f6

   1 Return-Path: <ethan.glasser.camp@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 8C408431FB6\r
   6         for <notmuch@notmuchmail.org>; Thu, 18 Oct 2012 22:15:37 -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: 1.7\r
  10 X-Spam-Level: *\r
  11 X-Spam-Status: No, score=1.7 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, FREEMAIL_REPLY=2.499, RCVD_IN_DNSWL_LOW=-0.7]\r
  14         autolearn=disabled\r
  15 Received: from olra.theworths.org ([127.0.0.1])\r
  16         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  17         with ESMTP id LAqMtJojAxur for <notmuch@notmuchmail.org>;\r
  18         Thu, 18 Oct 2012 22:15:37 -0700 (PDT)\r
  19 Received: from mail-vb0-f53.google.com (mail-vb0-f53.google.com\r
  20         [209.85.212.53]) (using TLSv1 with cipher RC4-SHA (128/128 bits))\r
  21         (No client certificate requested)\r
  22         by olra.theworths.org (Postfix) with ESMTPS id 0373A431FAE\r
  23         for <notmuch@notmuchmail.org>; Thu, 18 Oct 2012 22:15:36 -0700 (PDT)\r
  24 Received: by mail-vb0-f53.google.com with SMTP id fc21so81425vbb.26\r
  25         for <notmuch@notmuchmail.org>; Thu, 18 Oct 2012 22:15:36 -0700 (PDT)\r
  26 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113;\r
  27         h=from:to:subject:in-reply-to:references:user-agent:date:message-id\r
  28         :mime-version:content-type;\r
  29         bh=TG+0b7+IUvDAs6CSvkHQ4yxkmsaNavgoZKmEsptif5U=;\r
  30         b=h429bfY1oOvNJ1/4D8B7Du8Nbnr2aLTK2YCa2dSTuKXMUqQkLTvpD2a/N3qarNXdWY\r
  31         YNJOGv4AmZpMB5drdKz/quSswBmrd838aUHRdFlO+7sRZkQdHhAAXH7GaQjJCIrpBfKu\r
  32         GccOXynkhUgCynQNMl9dc2tosFB0NWYUd7O5nIIn9xqM6zszTRNwBgVLBPPEogyC6v/H\r
  33         IEYiEdMReQzzubd+5ZMgI7ZITjm3t1b17uNbYh8P72f6NIvMCLr3aNEz3p7qz/z6E9iR\r
  34         33OsvyQdE9VyHtao9vNy8N0i3eF+gHV2kLXYcXNSfasuslk6u2g3ewZjs3k+ZFqJ9K0o\r
  35         eDjg==\r
  36 Received: by 10.52.32.1 with SMTP id e1mr120027vdi.68.1350623735992;\r
  37         Thu, 18 Oct 2012 22:15:35 -0700 (PDT)\r
  38 Received: from smtp.gmail.com (p70-80.acedsl.com. [66.114.70.80])\r
  39         by mx.google.com with ESMTPS id xq19sm741776veb.0.2012.10.18.22.15.34\r
  40         (version=TLSv1/SSLv3 cipher=OTHER);\r
  41         Thu, 18 Oct 2012 22:15:35 -0700 (PDT)\r
  42 From: Ethan Glasser-Camp <ethan.glasser.camp@gmail.com>\r
  43 To: Peter Wang <novalazy@gmail.com>, notmuch@notmuchmail.org\r
  44 Subject: Re: [PATCH 7/8] lib: add NOTMUCH_EXCLUDE_FLAG to notmuch_exclude_t\r
  45 In-Reply-To: <1340198947-29370-8-git-send-email-novalazy@gmail.com>\r
  46 References: <1340198947-29370-1-git-send-email-novalazy@gmail.com>\r
  47         <1340198947-29370-8-git-send-email-novalazy@gmail.com>\r
  48 User-Agent: Notmuch/0.14+45~g6ea9330 (http://notmuchmail.org) Emacs/23.3.1\r
  49         (x86_64-pc-linux-gnu)\r
  50 Date: Fri, 19 Oct 2012 01:15:31 -0400\r
  51 Message-ID: <87txtr6o0c.fsf@betacantrips.com>\r
  52 MIME-Version: 1.0\r
  53 Content-Type: text/plain; charset=us-ascii\r
  54 X-BeenThere: notmuch@notmuchmail.org\r
  55 X-Mailman-Version: 2.1.13\r
  56 Precedence: list\r
  57 List-Id: "Use and development of the notmuch mail system."\r
  58         <notmuch.notmuchmail.org>\r
  59 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  60         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  61 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  62 List-Post: <mailto:notmuch@notmuchmail.org>\r
  63 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  64 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  65         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  66 X-List-Received-Date: Fri, 19 Oct 2012 05:15:38 -0000\r
  67 \r
  68 Peter Wang <novalazy@gmail.com> writes:\r
  69 \r
  70 > Add NOTMUCH_EXCLUDE_FLAG to notmuch_exclude_t so that it can\r
  71 > cover all four values of search --exclude in the cli.\r
  72 \r
  73 This series looks good to me. It's a nice clean up and a nice new\r
  74 feature. Patches all apply.\r
  75 \r
  76 However, I'm getting test failures like:\r
  77 \r
  78  FAIL   Search, exclude "deleted" messages from message search --exclude=false\r
  79         --- excludes.3.expected 2012-10-19 04:45:06.900518377 +0000\r
  80         +++ excludes.3.output   2012-10-19 04:45:06.900518377 +0000\r
  81         @@ -1,2 +1,2 @@\r
  82         -id:msg-001@notmuch-test-suite\r
  83          id:msg-002@notmuch-test-suite\r
  84         +id:msg-001@notmuch-test-suite\r
  85 \r
  86  FAIL   Search, don't exclude "deleted" messages when --exclude=flag specified\r
  87         --- excludes.7.expected 2012-10-19 04:45:07.004518378 +0000\r
  88         +++ excludes.7.output   2012-10-19 04:45:07.004518378 +0000\r
  89         @@ -1,2 +1,2 @@\r
  90         -thread:XXX   2001-01-05 [1/1] Notmuch Test Suite; Not deleted (inbox unread)\r
  91          thread:XXX   2001-01-05 [1/2] Notmuch Test Suite; Not deleted reply (deleted inbox unread)\r
  92         +thread:XXX   2001-01-05 [1/1] Notmuch Test Suite; Not deleted (inbox unread)\r
  93 \r
  94  FAIL   Search, don't exclude "deleted" messages from search if not configured\r
  95         --- excludes.8.expected 2012-10-19 04:45:07.028518377 +0000\r
  96         +++ excludes.8.output   2012-10-19 04:45:07.028518377 +0000\r
  97         @@ -1,2 +1,2 @@\r
  98         -thread:XXX   2001-01-05 [1/1] Notmuch Test Suite; Not deleted (inbox unread)\r
  99          thread:XXX   2001-01-05 [2/2] Notmuch Test Suite; Deleted (deleted inbox unread)\r
 100         +thread:XXX   2001-01-05 [1/1] Notmuch Test Suite; Not deleted (inbox unread)\r
 101 \r
 102 In other words, threads and messages are coming up out of order. I'm not\r
 103 sure of the right way to fix this. If you would like me to try sticking\r
 104 "| sort" here and there in the tests I will do so. I'm not sure if the\r
 105 test suite is guaranteed to scan messages in a certain order.\r
 106 \r
 107 Mark Walters wrote in\r
 108 id:"1340198947-29370-5-git-send-email-novalazy@gmail.com" that he\r
 109 thought patch 1/8 seemed more intrusive than he liked. Maybe I just have\r
 110 a higher standard for "intrusive" than he does ;) but I thought it was\r
 111 fine.\r
 112 \r
 113 It looks like you have better wording for patch 4/8 so I'd like to see\r
 114 you resend it.\r
 115 \r
 116 > -         if (query->omit_excluded != NOTMUCH_EXCLUDE_FALSE)\r
 117 > +         if (query->omit_excluded == NOTMUCH_EXCLUDE_TRUE ||\r
 118 > +             query->omit_excluded == NOTMUCH_EXCLUDE_ALL)\r
 119 > +         {\r
 120 >               final_query = Xapian::Query (Xapian::Query::OP_AND_NOT,\r
 121 >                                            final_query, exclude_query);\r
 122 > -         else {\r
 123 > +         } else {\r
 124 \r
 125 "House style" is to not put braces around one-line then-clauses. This is\r
 126 the only place where you did that.\r
 127 \r
 128 I'm marking patches 3, 4, and 7 as moreinfo. Please resubmit!\r
 129 \r
 130 Ethan\r