ad/1ac2f514b5390ebdc20b738763e22f9653b74a

   1 Return-Path: <cworth@cworth.org>\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 3F5CE431FBC\r
   6         for <notmuch@notmuchmail.org>; Wed, 24 Feb 2010 10:26:51 -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: -2.519\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-2.519 tagged_above=-999 required=5\r
  12         tests=[ALL_TRUSTED=-1.8, AWL=0.391, BAYES_05=-1.11] 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 ViLbhHeGDmsS; Wed, 24 Feb 2010 10:26:50 -0800 (PST)\r
  16 Received: from yoom.home.cworth.org (localhost [127.0.0.1])\r
  17         by olra.theworths.org (Postfix) with ESMTP id 7E323431FAE;\r
  18         Wed, 24 Feb 2010 10:26:50 -0800 (PST)\r
  19 Received: by yoom.home.cworth.org (Postfix, from userid 1000)\r
  20         id D51DB25427B; Wed, 24 Feb 2010 10:26:47 -0800 (PST)\r
  21 From: Carl Worth <cworth@cworth.org>\r
  22 To: Sebastian Spaeth <Sebastian@SSpaeth.de>, Jameson Rollins\r
  23         <jrollins@finestructure.net>, Notmuch Mail list <notmuch@notmuchmail.org>\r
  24 In-Reply-To: <87ljes0yy0.fsf@SSpaeth.de>\r
  25 References: <87ska1vh7r.fsf@servo.finestructure.net>\r
  26         <87ljes0yy0.fsf@SSpaeth.de>\r
  27 Date: Wed, 24 Feb 2010 10:26:47 -0800\r
  28 Message-ID: <87vddmwkuw.fsf@yoom.home.cworth.org>\r
  29 MIME-Version: 1.0\r
  30 Content-Type: multipart/signed; boundary="=-=-=";\r
  31         micalg=pgp-sha1; protocol="application/pgp-signature"\r
  32 Subject: Re: [notmuch] [PATCH] Simplify "unread" tag handling in emacs UI.\r
  33 X-BeenThere: notmuch@notmuchmail.org\r
  34 X-Mailman-Version: 2.1.13\r
  35 Precedence: list\r
  36 List-Id: "Use and development of the notmuch mail system."\r
  37         <notmuch.notmuchmail.org>\r
  38 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  39         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  40 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  41 List-Post: <mailto:notmuch@notmuchmail.org>\r
  42 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  43 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  44         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  45 X-List-Received-Date: Wed, 24 Feb 2010 18:26:51 -0000\r
  46 \r
  47 --=-=-=\r
  48 \r
  49 On Wed, 17 Feb 2010 14:33:11 +0100, "Sebastian Spaeth" <Sebastian@SSpaeth.de> wrote:\r
  50 > On Tue, 19 Jan 2010 17:54:16 -0500, Jameson Rollins <jrollins@finestructure.net> wrote:\r
  51 > > This patch is intended to greatly simplify the handling of the\r
  52 > > "unread" tag in the emacs UI.  This patch adds a new function\r
  53 > > 'notmuch-show-mark-read', that removes the "unread" tag in\r
  54 > > notmuch-show-mode.  This function is then executed as a\r
  55 > > notmuch-show-hook, and by notmuch-show-next-message.  All of the\r
  56 > > functions that explicitly marked messages as unread are removed or\r
  57 > > renamed.\r
  58 \r
  59 Hi Jameson,\r
  60 \r
  61 Thanks for contributing the patch. This exact feature, (removing all\r
  62 commands with "and mark read" in their names), has been on my todo list\r
  63 for too long, and I'm anxious to remove it from that. But...\r
  64 \r
  65 > It then checks the unread status in order to decide whether to proceed\r
  66 > to the next again. So with your patch notmuch-show-next-unread-message\r
  67 > will skip through all messages in a thread thinking they are all read\r
  68 > (and actually marking all as read).\r
  69 \r
  70 ...that seems like a fatal bug in this script. Thanks for noting that\r
  71 Sebastian.\r
  72 \r
  73 I'm very interested in augmenting the test suite such that we could\r
  74 write emacs-based tests that would capture this bug. It seems that with\r
  75 the --batch, --load, and --funcall options it shouldn't be too hard to\r
  76 write a modular function exercising emacs code and then execute it\r
  77 no-interactively. Then we could either inspect the state of the database\r
  78 to ensure the operation worked as desired, or else have the test\r
  79 function write a buffer out to a file and test that the file has the\r
  80 desired contents.\r
  81 \r
  82 So that's something I'll be working on soon. And of course, I'll always\r
  83 be glad to accept any help.\r
  84 \r
  85 -Carl\r
  86 \r
  87 --=-=-=\r
  88 Content-Type: application/pgp-signature\r
  89 \r
  90 -----BEGIN PGP SIGNATURE-----\r
  91 Version: GnuPG v1.4.10 (GNU/Linux)\r
  92 \r
  93 iD8DBQFLhW9n6JDdNq8qSWgRAg1SAJ42ckxe1mjQ8EGtTYY4F82oppri8gCeL8vx\r
  94 TAWYawtFdqm0VCPVvUzww/k=\r
  95 =3Ud/\r
  96 -----END PGP SIGNATURE-----\r
  97 --=-=-=--\r