00/3c511219344e44df184feb0115b764dc5a4aeb

   1 Return-Path: <Sebastian@SSpaeth.de>\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 ADCAD431FC0\r
   6         for <notmuch@notmuchmail.org>; Thu, 25 Feb 2010 02:23:49 -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: -0.661\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.661 tagged_above=-999 required=5\r
  12         tests=[AWL=-0.662, BAYES_50=0.001] autolearn=unavailable\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 3eOpaXIFEukG for <notmuch@notmuchmail.org>;\r
  16         Thu, 25 Feb 2010 02:23:48 -0800 (PST)\r
  17 Received: from homiemail-a19.g.dreamhost.com (caiajhbdcahe.dreamhost.com\r
  18         [208.97.132.74])\r
  19         by olra.theworths.org (Postfix) with ESMTP id CE0B2431FAE\r
  20         for <notmuch@notmuchmail.org>; Thu, 25 Feb 2010 02:23:48 -0800 (PST)\r
  21 Received: from sspaeth.de (unknown [195.190.178.84])\r
  22         by homiemail-a19.g.dreamhost.com (Postfix) with ESMTPA id 8BAD2604061; \r
  23         Thu, 25 Feb 2010 02:23:43 -0800 (PST)\r
  24 Received: by sspaeth.de (sSMTP sendmail emulation);\r
  25         Thu, 25 Feb 2010 11:23:41 +0100\r
  26 From: "Sebastian Spaeth" <Sebastian@SSpaeth.de>\r
  27 To: Jameson Rollins <jrollins@finestructure.net>, Carl Worth\r
  28         <cworth@cworth.org>, Notmuch Mail list <notmuch@notmuchmail.org>\r
  29 In-Reply-To: <87r5oa1lvo.fsf@servo.finestructure.net>\r
  30 References: <87ska1vh7r.fsf@servo.finestructure.net>\r
  31         <87ljes0yy0.fsf@SSpaeth.de> <87vddmwkuw.fsf@yoom.home.cworth.org>\r
  32         <87r5oa1lvo.fsf@servo.finestructure.net>\r
  33 Date: Thu, 25 Feb 2010 11:23:40 +0100\r
  34 Message-ID: <87hbp5bolv.fsf@SSpaeth.de>\r
  35 MIME-Version: 1.0\r
  36 Content-Type: text/plain; charset=utf-8\r
  37 Content-Transfer-Encoding: quoted-printable\r
  38 Subject: Re: [notmuch] [PATCH] Simplify "unread" tag handling in emacs UI.\r
  39 X-BeenThere: notmuch@notmuchmail.org\r
  40 X-Mailman-Version: 2.1.13\r
  41 Precedence: list\r
  42 List-Id: "Use and development of the notmuch mail system."\r
  43         <notmuch.notmuchmail.org>\r
  44 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  45         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  46 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  47 List-Post: <mailto:notmuch@notmuchmail.org>\r
  48 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  49 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  50         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  51 X-List-Received-Date: Thu, 25 Feb 2010 10:23:49 -0000\r
  52 \r
  53 On Wed, 24 Feb 2010 14:20:27 -0500, Jameson Rollins <jrollins@finestructure=\r
  54 .net> wrote:\r
  55 > On Wed, 24 Feb 2010 10:26:47 -0800, Carl Worth <cworth@cworth.org> wrote:\r
  56 \r
  57 > > > It then checks the unread status in order to decide whether to proceed\r
  58 > > > to the next again. So with your patch notmuch-show-next-unread-message\r
  59 > > > will skip through all messages in a thread thinking they are all read\r
  60 > > > (and actually marking all as read).\r
  61 > >=20\r
  62 > > ...that seems like a fatal bug in this script. Thanks for noting that\r
  63 > > Sebastian.\r
  64 >=20\r
  65 > I certainly don't see it as fatal, but it is something we should\r
  66 > resolve.  I think the simplification that the patch provides is worth\r
  67 > it.\r
  68 \r
  69 Well, it is fatal in the sense that it makes that function useless. But\r
  70 it's not as bad as it is currently unused. You will only notice it if you\r
  71 make the "skip to next unread" (interactive) and bind it to a key (which\r
  72 is what I have done). I still think it's worth taking this patch and fixing=\r
  73  it then.\r
  74 \r
  75 The solutions I can see are:=20\r
  76 - split into a "notmuch-show-next-message"\r
  77   and a notmuch-show-show-next-message (what a naming!) function.=20\r
  78   One would merely skip to the next and one actually show and mark unread.\r
  79 - Introduce a second argument option whether we should mark as unread or no=\r
  80 t.\r
  81 - rework "notmuch-show-next-unread-message" to not make use of\r
  82   "notmuch-show-next-message" but e.g. to just use\r
  83   "notmuch-show-find-next-message" or so.\r
  84 \r
  85 Does this make any sense?\r
  86 \r
  87 > I'm seeing the notmuch-show-next-unread-message as a non-interactive\r
  88 > function that's not currently called by any other functions, and is\r
  89 > therefore not being used.  Sebastian, are you using that in a private\r
  90 > function, or am I misreading the code?\r
  91 \r
  92 Touch=C3=A9. As explained above. I made it interactive and bound it to 'n'.\r
  93 \r
  94 Sebastian\r