00/68ba06080e97b7ea52fd40f62d1d1eb85c3bef

   1 Return-Path: <amdragon@mit.edu>\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 4EB0F431FB6\r
   6         for <notmuch@notmuchmail.org>; Thu,  5 Apr 2012 18:09:47 -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.7\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.7 tagged_above=-999 required=5\r
  12         tests=[RCVD_IN_DNSWL_LOW=-0.7] autolearn=disabled\r
  13 Received: from olra.theworths.org ([127.0.0.1])\r
  14         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  15         with ESMTP id hNwaU9bFtzxw for <notmuch@notmuchmail.org>;\r
  16         Thu,  5 Apr 2012 18:09:46 -0700 (PDT)\r
  17 Received: from dmz-mailsec-scanner-1.mit.edu (DMZ-MAILSEC-SCANNER-1.MIT.EDU\r
  18         [18.9.25.12])\r
  19         by olra.theworths.org (Postfix) with ESMTP id 7F62A431FAE\r
  20         for <notmuch@notmuchmail.org>; Thu,  5 Apr 2012 18:09:46 -0700 (PDT)\r
  21 X-AuditID: 1209190c-b7fad6d000000920-b7-4f7e42571a12\r
  22 Received: from mailhub-auth-3.mit.edu ( [18.9.21.43])\r
  23         by dmz-mailsec-scanner-1.mit.edu (Symantec Messaging Gateway) with SMTP\r
  24         id F6.1F.02336.7524E7F4; Thu,  5 Apr 2012 21:09:43 -0400 (EDT)\r
  25 Received: from outgoing.mit.edu (OUTGOING-AUTH.MIT.EDU [18.7.22.103])\r
  26         by mailhub-auth-3.mit.edu (8.13.8/8.9.2) with ESMTP id q3619g7J032473; \r
  27         Thu, 5 Apr 2012 21:09:43 -0400\r
  28 Received: from awakening.csail.mit.edu (awakening.csail.mit.edu [18.26.4.91])\r
  29         (authenticated bits=0)\r
  30         (User authenticated as amdragon@ATHENA.MIT.EDU)\r
  31         by outgoing.mit.edu (8.13.6/8.12.4) with ESMTP id q3619eHI013233\r
  32         (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=NOT);\r
  33         Thu, 5 Apr 2012 21:09:41 -0400 (EDT)\r
  34 Received: from amthrax by awakening.csail.mit.edu with local (Exim 4.77)\r
  35         (envelope-from <amdragon@mit.edu>)\r
  36         id 1SFxgK-0004Oq-JL; Thu, 05 Apr 2012 21:09:40 -0400\r
  37 From: Austin Clements <amdragon@MIT.EDU>\r
  38 To: Mark Walters <markwalters1009@gmail.com>, notmuch@notmuchmail.org\r
  39 Subject: Re: [PATCH v2 1/6] lib: change default for\r
  40         notmuch_query_set_omit_excluded\r
  41 In-Reply-To: <1333234442-28616-2-git-send-email-markwalters1009@gmail.com>\r
  42 References: <1333234442-28616-1-git-send-email-markwalters1009@gmail.com>\r
  43         <1333234442-28616-2-git-send-email-markwalters1009@gmail.com>User-Agent:\r
  44         Notmuch/0.11.1+358~g3b96caf (http://notmuchmail.org)\r
  45         Emacs/23.3.1 (i486-pc-linux-gnu)\r
  46 Date: Thu, 05 Apr 2012 21:09:40 -0400\r
  47 Message-ID: <8762ddd5t7.fsf@awakening.csail.mit.edu>\r
  48 MIME-Version: 1.0\r
  49 Content-Type: text/plain; charset=us-ascii\r
  50 X-Brightmail-Tracker:\r
  51  H4sIAAAAAAAAA+NgFnrIIsWRmVeSWpSXmKPExsUixCmqrRvuVOdvcHiGuMXquTwW12/OZHZg\r
  52         8tg56y67x7NVt5gDmKK4bFJSczLLUov07RK4MubPns5WcFymYtnGZ4wNjC3iXYycHBICJhIT\r
  53         J85lh7DFJC7cW8/WxcjFISSwj1Hi65deZghnPaPE7pV7WSCcE0wS2y92skI4SxglVv2ayAzS\r
  54         zyagIbFt/3JGEFtEwFXi6bfPYHFhgWCJjoaVQHEODk4BL4ntTwIhel8wSqzbcY4FpIZFQFXi\r
  55         xMJHrCA2L9BNe+9vZoSwBSVOznwCVsMsoCVx499LpgmM/LOQpGYhSS1gZFrFKJuSW6Wbm5iZ\r
  56         U5yarFucnJiXl1qka6iXm1mil5pSuokRHHqSPDsY3xxUOsQowMGoxMN7amutvxBrYllxZe4h\r
  57         RkkOJiVRXjaHOn8hvqT8lMqMxOKM+KLSnNTiQ4wSHMxKIrxP+YFyvCmJlVWpRfkwKWkOFiVx\r
  58         XhWtd35CAumJJanZqakFqUUwWRkODiUJ3reOQI2CRanpqRVpmTklCGkmDk6Q4TxAw++D1PAW\r
  59         FyTmFmemQ+RPMepyPD/Ue4VRiCUvPy9VSpx3KUiRAEhRRmke3BxYynjFKA70ljDvKZAqHmC6\r
  60         gZv0CmgJE9CSk1+rQZaUJCKkpBoYhe4ZrneNYp54yvlJvvRqY1ZLN6Z7IVrRvJJxK+Z8/HMm\r
  61         Ikt0WftXrXtHtSSWTXib+MdaYdHTeAbNkJhDBxzuMP94vaRqwZyOd9M2xzuaNbp2HDNv8bZh\r
  62         NPRc4rf3mWq38A2bE/oplWUVv+oDH9lLiAvdu+HRp5Hygkt/6fLWApYfP92SXvIpsRRnJBpq\r
  63         MRcVJwIAQLnwy/QCAAA=\r
  64 X-BeenThere: notmuch@notmuchmail.org\r
  65 X-Mailman-Version: 2.1.13\r
  66 Precedence: list\r
  67 List-Id: "Use and development of the notmuch mail system."\r
  68         <notmuch.notmuchmail.org>\r
  69 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  70         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  71 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  72 List-Post: <mailto:notmuch@notmuchmail.org>\r
  73 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  74 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  75         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  76 X-List-Received-Date: Fri, 06 Apr 2012 01:09:47 -0000\r
  77 \r
  78 On Sat, 31 Mar 2012, Mark Walters <markwalters1009@gmail.com> wrote:\r
  79 > ---\r
  80 >  lib/notmuch.h |   11 ++++++-----\r
  81 >  lib/query.cc  |   10 +++++-----\r
  82 >  2 files changed, 11 insertions(+), 10 deletions(-)\r
  83 >\r
  84 > diff --git a/lib/notmuch.h b/lib/notmuch.h\r
  85 > index babd208..029a2c3 100644\r
  86 > --- a/lib/notmuch.h\r
  87 > +++ b/lib/notmuch.h\r
  88 > @@ -449,12 +449,13 @@ typedef enum {\r
  89 >  const char *\r
  90 >  notmuch_query_get_query_string (notmuch_query_t *query);\r
  91 >  \r
  92 > -/* Specify whether to results should omit the excluded results rather\r
  93 > - * than just marking them excluded. This is useful for passing a\r
  94 > - * notmuch_messages_t not containing the excluded messages to other\r
  95 > - * functions. */\r
  96 > +/* Specify whether to omit the excluded results or just flag\r
  97 > + * them. Note when calling notmuch_query_search_threads, the returned\r
  98 > + * thread will contain all messages regardless of this setting but,\r
  99 > + * unless this is unset, only threads matching in a non-excluded\r
 100 > + * message will be returned. */\r
 101 \r
 102 The docs seem a bit awkward (I suppose the interface is slightly\r
 103 awkward, but the comment makes it sound much more so) and I don't see a\r
 104 mention of the default behavior anywhere.  How about something like\r
 105 \r
 106   Specify whether to omit excluded results or simply flag them.  By\r
 107   default, this is set to TRUE.\r
 108 \r
 109   If this is TRUE, notmuch_query_search_messages will omit excluded\r
 110   messages from the results.  notmuch_query_search_threads will omit\r
 111   threads that match only in excluded messages, but will include all\r
 112   messages in threads that are only partially excluded.\r
 113 \r
 114 It might be nice to mention the performance implications of the various\r
 115 options, too.\r
 116 \r
 117 >  void\r
 118 > -notmuch_query_set_omit_excluded_messages (notmuch_query_t *query, notmuch_bool_t omit);\r
 119 > +notmuch_query_set_omit_excluded_messages (notmuch_query_t *query, notmuch_bool_t omit_excluded);\r
 120 \r
 121 Should this be notmuch_query_set_omit_excluded?\r
 122 \r
 123 >  \r
 124 >  /* Specify the sorting desired for this query. */\r
 125 >  void\r
 126 > diff --git a/lib/query.cc b/lib/query.cc\r
 127 > index 68ac1e4..f7c7099 100644\r
 128 > --- a/lib/query.cc\r
 129 > +++ b/lib/query.cc\r
 130 > @@ -28,7 +28,7 @@ struct _notmuch_query {\r
 131 >      const char *query_string;\r
 132 >      notmuch_sort_t sort;\r
 133 >      notmuch_string_list_t *exclude_terms;\r
 134 > -    notmuch_bool_t omit_excluded_messages;\r
 135 > +    notmuch_bool_t omit_excluded;\r
 136 >  };\r
 137 >  \r
 138 >  typedef struct _notmuch_mset_messages {\r
 139 > @@ -92,7 +92,7 @@ notmuch_query_create (notmuch_database_t *notmuch,\r
 140 >  \r
 141 >      query->exclude_terms = _notmuch_string_list_create (query);\r
 142 >  \r
 143 > -    query->omit_excluded_messages = FALSE;\r
 144 > +    query->omit_excluded = TRUE;\r
 145 >  \r
 146 >      return query;\r
 147 >  }\r
 148 > @@ -104,9 +104,9 @@ notmuch_query_get_query_string (notmuch_query_t *query)\r
 149 >  }\r
 150 >  \r
 151 >  void\r
 152 > -notmuch_query_set_omit_excluded_messages (notmuch_query_t *query, notmuch_bool_t omit)\r
 153 > +notmuch_query_set_omit_excluded_messages (notmuch_query_t *query, notmuch_bool_t omit_excluded)\r
 154 >  {\r
 155 > -    query->omit_excluded_messages = omit;\r
 156 > +    query->omit_excluded = omit_excluded;\r
 157 >  }\r
 158 >  \r
 159 >  void\r
 160 > @@ -220,7 +220,7 @@ notmuch_query_search_messages (notmuch_query_t *query)\r
 161 >       if (query->exclude_terms) {\r
 162 >           exclude_query = _notmuch_exclude_tags (query, final_query);\r
 163 >  \r
 164 > -         if (query->omit_excluded_messages)\r
 165 > +         if (query->omit_excluded)\r
 166 >               final_query = Xapian::Query (Xapian::Query::OP_AND_NOT,\r
 167 >                                            final_query, exclude_query);\r
 168 >           else {\r
 169 > -- \r
 170 > 1.7.9.1\r
 171 >\r
 172 > _______________________________________________\r
 173 > notmuch mailing list\r
 174 > notmuch@notmuchmail.org\r
 175 > http://notmuchmail.org/mailman/listinfo/notmuch\r