58/5bdb862697238a77b5b9079f7928cd9ff78ef9

   1 Return-Path: <prvs=jrosenthal=70834ccb8@jhu.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 C207C431FC1\r
   6         for <notmuch@notmuchmail.org>; Fri,  9 Apr 2010 06:11:02 -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.301\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-2.301 tagged_above=-999 required=5\r
  12         tests=[BAYES_20=-0.001, RCVD_IN_DNSWL_MED=-2.3] autolearn=ham\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 t4IypQPjmAgZ for <notmuch@notmuchmail.org>;\r
  16         Fri,  9 Apr 2010 06:11:01 -0700 (PDT)\r
  17 Received: from ipex2.johnshopkins.edu (ipex2.johnshopkins.edu [162.129.8.151])\r
  18         by olra.theworths.org (Postfix) with ESMTP id B15F44196F2\r
  19         for <notmuch@notmuchmail.org>; Fri,  9 Apr 2010 06:11:01 -0700 (PDT)\r
  20 X-IronPort-AV: E=Sophos;i="4.52,177,1270440000"; d="scan'208";a="319959808"\r
  21 Received: from unknown (HELO gogo) ([12.236.65.28])\r
  22         by ipex2.johnshopkins.edu with ESMTP/TLS/AES256-SHA;\r
  23         09 Apr 2010 09:11:01 -0400\r
  24 Received: from jkr by gogo with local (Exim 4.69)\r
  25         (envelope-from <jrosenthal@jhu.edu>)\r
  26         id 1O0EvE-0000h8-Ru; Fri, 09 Apr 2010 10:11:00 -0400\r
  27 From: Jesse Rosenthal <jrosenthal@jhu.edu>\r
  28 To: Carl Worth <cworth@cworth.org>, notmuch@notmuchmail.org\r
  29 Subject: Re: [notmuch] [PATCH v2] notmuch.el: add functionality in notmuch\r
  30         search mode to add or remove tags by region\r
  31 In-Reply-To: <87aatfq88h.fsf@yoom.home.cworth.org>\r
  32 References: <m1636w7au5.fsf@watt.gilman.jhu.edu> <87sk90ragj.fsf@jhu.edu>\r
  33         <87aatfq88h.fsf@yoom.home.cworth.org>\r
  34 Date: Fri, 09 Apr 2010 10:11:00 -0400\r
  35 Message-ID: <877hogaf7v.fsf@jhu.edu>\r
  36 MIME-Version: 1.0\r
  37 Content-Type: text/plain; charset=us-ascii\r
  38 X-BeenThere: notmuch@notmuchmail.org\r
  39 X-Mailman-Version: 2.1.13\r
  40 Precedence: list\r
  41 List-Id: "Use and development of the notmuch mail system."\r
  42         <notmuch.notmuchmail.org>\r
  43 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  44         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  45 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  46 List-Post: <mailto:notmuch@notmuchmail.org>\r
  47 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  48 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  49         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  50 X-List-Received-Date: Fri, 09 Apr 2010 13:11:02 -0000\r
  51 \r
  52 On Wed, 07 Apr 2010 14:10:38 -0700, Carl Worth <cworth@cworth.org> wrote:\r
  53 > On Tue, 16 Feb 2010 19:07:40 -0500, Jesse Rosenthal <jrosenthal@jhu.edu> wrote:\r
  54 > I think this feature is very useful, and that the region is definitely\r
  55 > an appropriate way to implement it, (doing region-based operations is\r
  56 > very natural for emacs users). Mutt-style marking could be implemented\r
  57 > as well, but that would be separate I think.\r
  58 \r
  59 Great -- never sure if my intuitive use corresponds with that of others.\r
  60 \r
  61 > I also don't like the duplication of code in\r
  62 > notmuch-search-add-tag-thread and notmuch-search-add-tag-region, (and\r
  63 > the same in the remove case). Fortunately, I think this easy to avoid by\r
  64 > simply making notmuch-search-add-tag-thread call:\r
  65 > \r
  66 >        (notmuch-search-add-tag-region tag (point) (point))\r
  67 > \r
  68 > and the same in the remove case.\r
  69 \r
  70 Good idea.\r
  71 \r
  72 > \r
  73 > But I haven't pushed this patch yet for a flaw in the case of selecting\r
  74 > beyond the last thread, (such as selecting to the line that includes the\r
  75 > "End of search results). If I try an operation on that line, it will act\r
  76 > like it works, (it displays the new tags by all threads), but then a\r
  77 > refresh makes them all disappear again. Presumably the "notmuch tag"\r
  78 > command is failing, this isn't being noticed, and the code marches on to\r
  79 > update the representation of the tags.\r
  80 \r
  81 Good point, and shouldn't be too hard to fix. It seems like it could be\r
  82 done either (a) at the lisp level (filtering out the blank entries), or\r
  83 (b) at the literal buffer level (having a step that sets point-max for\r
  84 the region appropriately). The choice (a) seems clearer in an abstract\r
  85 way, but has the downside of needing to be repeated twice (both for\r
  86 database tagging, and for rewriting the tags on the screen). I'll play\r
  87 with both and see which is clearer.\r
  88 \r
  89 Best,\r
  90 Jesse\r