77/1682b97c5d1a2753569cd50bdd1a2c3a783091

   1 Return-Path: <jrollins@finestructure.net>\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 E9504431FBD\r
   6         for <notmuch@notmuchmail.org>; Sat, 14 Apr 2012 12:08:00 -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: -2.29\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-2.29 tagged_above=-999 required=5\r
  12         tests=[RCVD_IN_DNSWL_MED=-2.3, T_MIME_NO_TEXT=0.01] 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 1ObXVJzWH16d for <notmuch@notmuchmail.org>;\r
  16         Sat, 14 Apr 2012 12:08:00 -0700 (PDT)\r
  17 Received: from outgoing-mail.its.caltech.edu (outgoing-mail.its.caltech.edu\r
  18         [131.215.239.19])\r
  19         by olra.theworths.org (Postfix) with ESMTP id 8008A431FAF\r
  20         for <notmuch@notmuchmail.org>; Sat, 14 Apr 2012 12:08:00 -0700 (PDT)\r
  21 Received: from fire-doxen.imss.caltech.edu (localhost [127.0.0.1])\r
  22         by fire-doxen-postvirus (Postfix) with ESMTP id 315942E50D97;\r
  23         Sat, 14 Apr 2012 12:08:00 -0700 (PDT)\r
  24 X-Spam-Scanned: at Caltech-IMSS on fire-doxen by amavisd-new\r
  25 Received: from finestructure.net (unknown [76.89.193.65])\r
  26         (Authenticated sender: jrollins)\r
  27         by fire-doxen-submit (Postfix) with ESMTP id 6088B2E50D8C;\r
  28         Sat, 14 Apr 2012 12:07:57 -0700 (PDT)\r
  29 Received: by finestructure.net (Postfix, from userid 1000)\r
  30         id EF53745D; Sat, 14 Apr 2012 12:07:56 -0700 (PDT)\r
  31 From: Jameson Graef Rollins <jrollins@finestructure.net>\r
  32 To: Mark Walters <markwalters1009@gmail.com>,\r
  33         Notmuch Mail <notmuch@notmuchmail.org>\r
  34 Subject: Re: [PATCH 5/8] emacs: modify search tag functions to use new\r
  35         notmuch-tag interface\r
  36 In-Reply-To: <871unwn40g.fsf@qmul.ac.uk>\r
  37 References: <1333354853-25729-1-git-send-email-jrollins@finestructure.net>\r
  38         <1333845338-22960-1-git-send-email-jrollins@finestructure.net>\r
  39         <1333845338-22960-2-git-send-email-jrollins@finestructure.net>\r
  40         <1333845338-22960-3-git-send-email-jrollins@finestructure.net>\r
  41         <1333845338-22960-4-git-send-email-jrollins@finestructure.net>\r
  42         <1333845338-22960-5-git-send-email-jrollins@finestructure.net>\r
  43         <1333845338-22960-6-git-send-email-jrollins@finestructure.net>\r
  44         <871unwn40g.fsf@qmul.ac.uk>\r
  45 User-Agent: Notmuch/0.12+112~gffb05d6 (http://notmuchmail.org) Emacs/23.4.1\r
  46         (x86_64-pc-linux-gnu)\r
  47 Date: Sat, 14 Apr 2012 12:07:54 -0700\r
  48 Message-ID: <871unq169h.fsf@servo.finestructure.net>\r
  49 MIME-Version: 1.0\r
  50 Content-Type: multipart/signed; boundary="=-=-=";\r
  51         micalg=pgp-sha256; protocol="application/pgp-signature"\r
  52 X-BeenThere: notmuch@notmuchmail.org\r
  53 X-Mailman-Version: 2.1.13\r
  54 Precedence: list\r
  55 List-Id: "Use and development of the notmuch mail system."\r
  56         <notmuch.notmuchmail.org>\r
  57 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  58         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  59 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  60 List-Post: <mailto:notmuch@notmuchmail.org>\r
  61 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  62 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  63         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  64 X-List-Received-Date: Sat, 14 Apr 2012 19:08:01 -0000\r
  65 \r
  66 --=-=-=\r
  67 \r
  68 On Mon, Apr 09 2012, Mark Walters <markwalters1009@gmail.com> wrote:\r
  69 > My only comment here is that I wonder about the tag-all command. I can\r
  70 > see that this is a performance win over applying tag region to the whole\r
  71 > buffer but is it another place where we have a race with incoming mail\r
  72 > (ie we tag everything that now matches rather than everything in the\r
  73 > buffer).\r
  74 \r
  75 Yes, I think this is a problem.  notmuch-{search,show}-tag-all are both\r
  76 subject to race conditions.  I suggest we remove notmuch-search-tag-all\r
  77 in favor of the region tagging capabilities of notmuch-show-tag.  I will\r
  78 submit a patch for that.  notmuch-show-tag-all is trickier, since there\r
  79 no tag-by-region capability at the moment.  That will have to be added\r
  80 before this function can be replaced.\r
  81 \r
  82 > A second advantage with the tag-region form is that it automatically\r
  83 > does excludes. Currently notmuch-tag does not do excludes as it looked\r
  84 > very hairy to implement correctly: thus the tag-all command could spend\r
  85 > a lot of time tagging excluded messages.\r
  86 \r
  87 Actually, I don't think this is true, at least for search mode.  In\r
  88 search mode threads are tagged by their thread id, therefore any\r
  89 excluded messages in the thread will receive the same tagging\r
  90 operations.\r
  91 \r
  92 The current operation is at least, in my opinion, self consistent.  If\r
  93 we're going to tag by thread then all messages in the thread should be\r
  94 tagged, whether or not they're excluded.  I do think, however, that it\r
  95 might be nice to add a tagging function that only tags the matching\r
  96 messages in the thread.  In that case excluded messages would not be\r
  97 tagged.  That's for future work, though.\r
  98 \r
  99 jamie.\r
 100 \r
 101 --=-=-=\r
 102 Content-Type: application/pgp-signature\r
 103 \r
 104 -----BEGIN PGP SIGNATURE-----\r
 105 Version: GnuPG v1.4.12 (GNU/Linux)\r
 106 \r
 107 iQIcBAEBCAAGBQJPicsKAAoJEO00zqvie6q8cZEQALmcO+ZmVhRNpyJBxPVldp9E\r
 108 LI+uT+DJNAHhaH5M+WuJDBJmFl232uG8wFrhcmwoLgK2WvJx8LmU4W9YGAafwgEz\r
 109 5HC8OoS3ybc6XvdALZRAoyvy/2FFboNN2tEbO9B7tBPNhyuDRs4iH6ZmCu74kMsA\r
 110 PztaFT7kPC7+zuiS7TxWxShLvnh/0rw9TrOvyW2L4031F9XeuphyUvWkSeJjRx8w\r
 111 HB69URfnaQBmP4PbNZz9ShiIOIMle5WPtqlpcxymcNg6wX05XxaLqGdJeKVrILLc\r
 112 TyZW1/rXY1YWicO0KRJ4GaaEDXjrsRrlxKXXyUvvMhUWoGVMlWQiUv2CK7VwltKb\r
 113 ecIgb1fmc1q03VPbg+HSOmivMcBMA+2iwBN55q0PHCy112y0pFl3dqOOy59EWKfg\r
 114 2yVqHh2aqXE7SK1dpqcZCQCeTWX092fKCMP5UsgNkqLR2jUtCz3bdy0t+N4/6XE0\r
 115 /NXtfOHeCVF2EZkImmoDmQQBqYk1X3x0dnKMT9VyHt/JzMuZJJQtRyB7l34KWw34\r
 116 jxaQPA5uqbdE9ZQdzfC6G0EcKZ2UZDccgf/ALrkfx+JG0kJjcr7NxRXpZVEWzz2F\r
 117 wnY9d5+gUBr7VOG052KPmtGCojOKECQNmfXHgjv+fRJIYjUHWT/iVFL7NCXq0VtZ\r
 118 164UfBKVzATHxe5WKFqc\r
 119 =q9b+\r
 120 -----END PGP SIGNATURE-----\r
 121 --=-=-=--\r