84/a178732b7d517f9dce0ee42e298efb1db54858

   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 03196431FAE\r
   6         for <notmuch@notmuchmail.org>; Wed, 24 Feb 2010 11:20:36 -0800 (PST)\r
   7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: -4.233\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-4.233 tagged_above=-999 required=5\r
  12         tests=[AWL=-0.048, BAYES_40=-0.185, RCVD_IN_DNSWL_MED=-4]\r
  13         autolearn=unavailable\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 QkIrycad-cJs for <notmuch@notmuchmail.org>;\r
  17         Wed, 24 Feb 2010 11:20:33 -0800 (PST)\r
  18 Received: from brinza.cc.columbia.edu (brinza.cc.columbia.edu [128.59.29.8])\r
  19         by olra.theworths.org (Postfix) with ESMTP id CA6F3431FBC\r
  20         for <notmuch@notmuchmail.org>; Wed, 24 Feb 2010 11:20:33 -0800 (PST)\r
  21 Received: from servo.finestructure.net (geco.phys.columbia.edu\r
  22         [128.59.170.159])\r
  23         (user=jgr2110 author=jrollins@finestructure.net mech=PLAIN bits=0)\r
  24         by brinza.cc.columbia.edu (8.14.3/8.14.3) with ESMTP id o1OJKUHP009853\r
  25         (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=NOT);\r
  26         Wed, 24 Feb 2010 14:20:30 -0500 (EST)\r
  27 Received: from jrollins by servo.finestructure.net with local (Exim 4.71)\r
  28         (envelope-from <jrollins@finestructure.net>)\r
  29         id 1NkMmc-0002Mp-BL; Wed, 24 Feb 2010 14:20:30 -0500\r
  30 From: Jameson Rollins <jrollins@finestructure.net>\r
  31 To: Carl Worth <cworth@cworth.org>, Sebastian Spaeth <Sebastian@SSpaeth.de>,\r
  32         Notmuch Mail list <notmuch@notmuchmail.org>\r
  33 In-Reply-To: <87vddmwkuw.fsf@yoom.home.cworth.org>\r
  34 References: <87ska1vh7r.fsf@servo.finestructure.net>\r
  35         <87ljes0yy0.fsf@SSpaeth.de> <87vddmwkuw.fsf@yoom.home.cworth.org>\r
  36 Date: Wed, 24 Feb 2010 14:20:27 -0500\r
  37 Message-ID: <87r5oa1lvo.fsf@servo.finestructure.net>\r
  38 MIME-Version: 1.0\r
  39 Content-Type: multipart/signed; boundary="=-=-=";\r
  40         micalg=pgp-sha256; protocol="application/pgp-signature"\r
  41 X-No-Spam-Score: Local\r
  42 X-Scanned-By: MIMEDefang 2.68 on 128.59.29.8\r
  43 Subject: Re: [notmuch] [PATCH] Simplify "unread" tag handling in emacs UI.\r
  44 X-BeenThere: notmuch@notmuchmail.org\r
  45 X-Mailman-Version: 2.1.13\r
  46 Precedence: list\r
  47 List-Id: "Use and development of the notmuch mail system."\r
  48         <notmuch.notmuchmail.org>\r
  49 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  50         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  51 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  52 List-Post: <mailto:notmuch@notmuchmail.org>\r
  53 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  54 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  55         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  56 X-List-Received-Date: Wed, 24 Feb 2010 19:20:36 -0000\r
  57 \r
  58 --=-=-=\r
  59 Content-Transfer-Encoding: quoted-printable\r
  60 \r
  61 On Wed, 24 Feb 2010 10:26:47 -0800, Carl Worth <cworth@cworth.org> wrote:\r
  62 > On Wed, 17 Feb 2010 14:33:11 +0100, "Sebastian Spaeth" <Sebastian@SSpaeth=\r
  63 .de> wrote:\r
  64 > > On Tue, 19 Jan 2010 17:54:16 -0500, Jameson Rollins <jrollins@finestruc=\r
  65 ture.net> wrote:\r
  66 > > > This patch is intended to greatly simplify the handling of the\r
  67 > > > "unread" tag in the emacs UI.  This patch adds a new function\r
  68 > > > 'notmuch-show-mark-read', that removes the "unread" tag in\r
  69 > > > notmuch-show-mode.  This function is then executed as a\r
  70 > > > notmuch-show-hook, and by notmuch-show-next-message.  All of the\r
  71 > > > functions that explicitly marked messages as unread are removed or\r
  72 > > > renamed.\r
  73 \r
  74 > Thanks for contributing the patch. This exact feature, (removing all\r
  75 > commands with "and mark read" in their names), has been on my todo list\r
  76 > for too long, and I'm anxious to remove it from that. But...\r
  77 >\r
  78 > > It then checks the unread status in order to decide whether to proceed\r
  79 > > to the next again. So with your patch notmuch-show-next-unread-message\r
  80 > > will skip through all messages in a thread thinking they are all read\r
  81 > > (and actually marking all as read).\r
  82 >=20\r
  83 > ...that seems like a fatal bug in this script. Thanks for noting that\r
  84 > Sebastian.\r
  85 \r
  86 I certainly don't see it as fatal, but it is something we should\r
  87 resolve.  I think the simplification that the patch provides is worth\r
  88 it.\r
  89 \r
  90 I'm seeing the notmuch-show-next-unread-message as a non-interactive\r
  91 function that's not currently called by any other functions, and is\r
  92 therefore not being used.  Sebastian, are you using that in a private\r
  93 function, or am I misreading the code?\r
  94 \r
  95 jamie.\r
  96 \r
  97 --=-=-=\r
  98 Content-Type: application/pgp-signature\r
  99 \r
 100 -----BEGIN PGP SIGNATURE-----\r
 101 Version: GnuPG v1.4.10 (GNU/Linux)\r
 102 \r
 103 iQIcBAEBCAAGBQJLhXv7AAoJEO00zqvie6q8etkQAIeR17fLFhIYfkCAxP7OasPY\r
 104 e3WK+wNNH5BnVF3OgmavBz6wl+Nwt8406RNDAgqS0c9SFQlXhLpVCXRJESUeRSsA\r
 105 fG+WmIqsC8EG/Gh7ApAnxiXL1IZ8wSvEfrQYCLudjqUwmCNthEHRyCEr5l5qcbCf\r
 106 ZWiE8XB74vvPLg+C5hUp074Vr7B3gYEDZXv3d8T7Et8DsqgIbjX95taxskSj4l69\r
 107 G1IRUVg29laTNGBKYu6xoYVpnfkzSmJHC70rIgGR/Vh4YSIRhHnTbamKB63ficrS\r
 108 IhcPTmse44JFrAqGASiw1ldbsjnq9V0Hy9HDZ8iyc4R+quJHXDxYwmOIQbvqBFnh\r
 109 Q2rTNoLwS/mBjGLJNmiJx3nvt383g5q9eFEbEFeIQ3Xa2ZWXEAcf6ttbITjociQQ\r
 110 8WyWDUmfpZ+ztwdJhEpW3iCyOjGucjMm+hhiS+EJdiSWfnTR6ZdSXISP2AUiW8cH\r
 111 xdg9pIYR34hSTInDY1mr54R5o4aIB+A4Pr9adZ8OwitiVp1foEAQxpFGgrvFtUp6\r
 112 JZkT7fMsiV+YxxvQR0ObHTadbyScc8Tb4IRkXbQhOzsG6iFoyopZXvVXMMb0rdC3\r
 113 iPz2xhThliLqRrqf6nJWFKNKdRvVHuXqLMr2fF3IiD8NxMbwjriaqxXySRDRfuKl\r
 114 r93KAJk6hQjm46APgGNG\r
 115 =Y0ut\r
 116 -----END PGP SIGNATURE-----\r
 117 --=-=-=--\r