95/a3d87867f139160a416525288858e32e2ecd2c

   1 Return-Path: <amdragon@mit.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 B584E431FAF\r
   6         for <notmuch@notmuchmail.org>; Thu,  2 Aug 2012 07:22:19 -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: -0.7\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.7 tagged_above=-999 required=5\r
  12         tests=[RCVD_IN_DNSWL_LOW=-0.7] 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 08LuxdKi2KLy for <notmuch@notmuchmail.org>;\r
  16         Thu,  2 Aug 2012 07:22:19 -0700 (PDT)\r
  17 Received: from dmz-mailsec-scanner-7.mit.edu (DMZ-MAILSEC-SCANNER-7.MIT.EDU\r
  18         [18.7.68.36])\r
  19         by olra.theworths.org (Postfix) with ESMTP id D1C70431FAE\r
  20         for <notmuch@notmuchmail.org>; Thu,  2 Aug 2012 07:22:18 -0700 (PDT)\r
  21 X-AuditID: 12074424-b7fa06d0000008c2-62-501a8d198c19\r
  22 Received: from mailhub-auth-3.mit.edu ( [18.9.21.43])\r
  23         by dmz-mailsec-scanner-7.mit.edu (Symantec Messaging Gateway) with SMTP\r
  24         id A1.30.02242.91D8A105; Thu,  2 Aug 2012 10:22:18 -0400 (EDT)\r
  25 Received: from outgoing.mit.edu (OUTGOING-AUTH.MIT.EDU [18.7.22.103])\r
  26         by mailhub-auth-3.mit.edu (8.13.8/8.9.2) with ESMTP id q72EMGCZ029728; \r
  27         Thu, 2 Aug 2012 10:22:16 -0400\r
  28 Received: from awakening.csail.mit.edu (awakening.csail.mit.edu [18.26.4.91])\r
  29         (authenticated bits=0)\r
  30         (User authenticated as amdragon@ATHENA.MIT.EDU)\r
  31         by outgoing.mit.edu (8.13.6/8.12.4) with ESMTP id q72EMEj3010218\r
  32         (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=NOT);\r
  33         Thu, 2 Aug 2012 10:22:15 -0400 (EDT)\r
  34 Received: from amthrax by awakening.csail.mit.edu with local (Exim 4.77)\r
  35         (envelope-from <amdragon@mit.edu>)\r
  36         id 1SwwI2-000507-JO; Thu, 02 Aug 2012 10:22:14 -0400\r
  37 From: Austin Clements <amdragon@MIT.EDU>\r
  38 To: Mark Walters <markwalters1009@gmail.com>, Jani Nikula <jani@nikula.org>,\r
  39         notmuch@notmuchmail.org\r
  40 Subject: Re: [PATCH] emacs: fix a bug introduced by the recent search\r
  41  cleanups.\r
  42 In-Reply-To: <87r4rpsquu.fsf@qmul.ac.uk>\r
  43 References: <1341354059-29396-1-git-send-email-amdragon@mit.edu>\r
  44         <1342892232-5659-1-git-send-email-amdragon@mit.edu>\r
  45         <1342892232-5659-8-git-send-email-amdragon@mit.edu>\r
  46         <87mx2dais8.fsf@nikula.org> <87r4rpsquu.fsf@qmul.ac.uk>\r
  47 User-Agent: Notmuch/0.13.2+59~g78b98ba (http://notmuchmail.org) Emacs/23.3.1\r
  48         (i486-pc-linux-gnu)\r
  49 Date: Thu, 02 Aug 2012 10:22:14 -0400\r
  50 Message-ID: <87y5lxnzl5.fsf@awakening.csail.mit.edu>\r
  51 MIME-Version: 1.0\r
  52 Content-Type: text/plain; charset=us-ascii\r
  53 X-Brightmail-Tracker:\r
  54  H4sIAAAAAAAAA+NgFvrDIsWRmVeSWpSXmKPExsUixCmqrSvVKxVgsOsnk0XTdGeLPfu8LFbP\r
  55         5bG4fnMmswOLx93TXB47Z91l97h1/zW7x7NVt5gDWKK4bFJSczLLUov07RK4MnY9fspcsFuk\r
  56         Yt+fWUwNjHcEuhg5OSQETCT+/T3KDmGLSVy4t56ti5GLQ0hgH6PEzGcnWCGc9YwSq5etZIRw\r
  57         TjBJPF5yAKpsCaPEzp9dzCD9bAIaEtv2L2cEsUUEciXmvJ3NCmIzC5hJNHftYuli5OAQFvCX\r
  58         eHg2EyTMCVR+qvEjE4gtJPCAUaK5wx2kRFQgQeLg8RqQMIuAqsTTJ+/YQGxeoEt33P/HCmEL\r
  59         Spyc+YQFYrqWxI1/L5kmMArOQpKahSS1gJFpFaNsSm6Vbm5iZk5xarJucXJiXl5qka65Xm5m\r
  60         iV5qSukmRlAgs7uo7GBsPqR0iFGAg1GJh3dzqWSAEGtiWXFl7iFGSQ4mJVHe911SAUJ8Sfkp\r
  61         lRmJxRnxRaU5qcWHGCU4mJVEeO/HAuV4UxIrq1KL8mFS0hwsSuK811Nu+gsJpCeWpGanphak\r
  62         FsFkZTg4lCR483uAGgWLUtNTK9Iyc0oQ0kwcnCDDeYCGh4HU8BYXJOYWZ6ZD5E8xKkqJ884C\r
  63         SQiAJDJK8+B6YYnmFaM40CvCvAUgVTzAJAXX/QpoMBPQYDszsMEliQgpqQZG0/1+U+LmNhS6\r
  64         FsgtvXdDJ2+P47pnf19HLwoO04m4Htcy5drUyr+T/k1UL6z99fw8jxaz6FdNBQud1YEJZsfV\r
  65         q8X6Dk9fLLf0dvp0j3/vWNn+/t22tOyGWGh2rdjrhQ6z/QJFXiouur3xYOsfrnrmn6++38p6\r
  66         vPPaqaNfEuUOdrF9EVods/TqdCWW4oxEQy3mouJEAK5GIM0PAwAA\r
  67 X-BeenThere: notmuch@notmuchmail.org\r
  68 X-Mailman-Version: 2.1.13\r
  69 Precedence: list\r
  70 List-Id: "Use and development of the notmuch mail system."\r
  71         <notmuch.notmuchmail.org>\r
  72 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  73         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  74 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  75 List-Post: <mailto:notmuch@notmuchmail.org>\r
  76 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  77 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  78         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  79 X-List-Received-Date: Thu, 02 Aug 2012 14:22:19 -0000\r
  80 \r
  81 On Thu, 02 Aug 2012, Mark Walters <markwalters1009@gmail.com> wrote:\r
  82 > In commit 5d0883e the function notmuch-search-next-thread was changed.\r
  83 > In particular it only goes to the next message if there is a next\r
  84 > message. This breaks notmuch-show-archive-thread-then-next. Fix this\r
  85 > by going to the "next" message whenever we are on a current message.\r
  86 > ---\r
  87 >\r
  88 >> On Sat, 21 Jul 2012, Austin Clements <amdragon@MIT.EDU> wrote:\r
  89 >>> At this point, the only remaining functions that don't support\r
  90 >>> multi-line search result formats are the thread navigation functions.\r
  91 >>> This patch fixes that by rewriting them in terms of\r
  92 >>> notmuch-search-result-{beginning,end}.\r
  93 >>>\r
  94 >>> This changes the behavior of notmuch-search-previous-thread slightly\r
  95 >>> so that if point isn't at the beginning of a result, it first moves\r
  96 >>> point to the beginning of the result.\r
  97 >>\r
  98 >> Hi Austin, bisecting suggests this patch (committed as 5d0883e) breaks\r
  99 >> notmuch-show-archive-thread-then-next and subsequently\r
 100 >> notmuch-show-advance-and-archive for me. When showing the last thread in\r
 101 >> the search results, notmuch-show-archive-thread-then-next used to exit\r
 102 >> to the end of the search buffer. Now it redisplays the current\r
 103 >> thread. When I hit SPC to cruise through mails, there's no indication\r
 104 >> that I've reached the end of search results, other than the same thread\r
 105 >> being displayed over and over.\r
 106 >\r
 107 > This might be the correct fix but I am not certain. It doesn't instantly\r
 108 > break for me! (It looks more similar to what happened before the parent\r
 109 > patch.)\r
 110 \r
 111 LGTM.  Technically this is still different from the old code, since it\r
 112 won't move forward once point is on the "End of search results" line.  I\r
 113 can't imagine anything actually depending on that (maybe something that\r
 114 loops until eobp?), but if we want to be paranoid, we could\r
 115 \r
 116 (if (notmuch-search-get-result)\r
 117     (goto-char (notmuch-search-result-end)\r
 118   (forward-line 1)))\r
 119 \r
 120 > Best wishes\r
 121 >\r
 122 > Mark\r
 123 >\r
 124 >\r
 125 >  emacs/notmuch.el |    2 +-\r
 126 >  1 files changed, 1 insertions(+), 1 deletions(-)\r
 127 >\r
 128 > diff --git a/emacs/notmuch.el b/emacs/notmuch.el\r
 129 > index fd1836f..d2d82a9 100644\r
 130 > --- a/emacs/notmuch.el\r
 131 > +++ b/emacs/notmuch.el\r
 132 > @@ -293,7 +293,7 @@ For a mouse binding, return nil."\r
 133 >  (defun notmuch-search-next-thread ()\r
 134 >    "Select the next thread in the search results."\r
 135 >    (interactive)\r
 136 > -  (when (notmuch-search-get-result (notmuch-search-result-end))\r
 137 > +  (when (notmuch-search-get-result)\r
 138 >      (goto-char (notmuch-search-result-end))))\r
 139 >  \r
 140 >  (defun notmuch-search-previous-thread ()\r
 141 > -- \r
 142 > 1.7.9.1\r