72/84d188b5af63484deb1232eeb4b986946b70b7

   1 Return-Path: <jani@nikula.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 C4EA3431FAF\r
   6         for <notmuch@notmuchmail.org>; Wed,  1 Aug 2012 23:51:27 -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 3ff89FoTiecm for <notmuch@notmuchmail.org>;\r
  16         Wed,  1 Aug 2012 23:51:25 -0700 (PDT)\r
  17 Received: from mail-gh0-f181.google.com (mail-gh0-f181.google.com\r
  18         [209.85.160.181]) (using TLSv1 with cipher RC4-SHA (128/128 bits))\r
  19         (No client certificate requested)\r
  20         by olra.theworths.org (Postfix) with ESMTPS id C0814431FAE\r
  21         for <notmuch@notmuchmail.org>; Wed,  1 Aug 2012 23:51:25 -0700 (PDT)\r
  22 Received: by ghbz13 with SMTP id z13so10091945ghb.26\r
  23         for <notmuch@notmuchmail.org>; Wed, 01 Aug 2012 23:51:25 -0700 (PDT)\r
  24 X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;\r
  25         d=google.com; s=20120113;\r
  26         h=from:to:cc:subject:in-reply-to:references:user-agent:date\r
  27         :message-id:mime-version:content-type:x-gm-message-state;\r
  28         bh=wj9IaBXD21l2qSjDBSvM4onXeJzOG3EpcFJpylIM9m4=;\r
  29         b=PVxfvuDNlTi2XEA0STA8b4Y6tExiBXq4TxVBqCGtkyD/rdCxSvl+Cgs9PAkvCJGMyJ\r
  30         6cZ834B7SI34bR0Ab05IrqZvu5q2QBkwQCYoP1o8TrJZYkD6hXE7WhusyLyYGzIOknTw\r
  31         RLKYgt+ssh9tokZjY1YM7VhsBQDfQWXKsN8Ck2oQHgUBpY/7OxltJwbi5b5pgRy7Z/GX\r
  32         bChRGZ85luFb2XJ3F5uU2FhQfdqH3T6afUPNzsCCmnYK2m0rEldhSDwp2lH/fMxSGz9D\r
  33         FmLO6tdUhR8H24K5UBtPekmlaVLsPMcpEiPY9S3mvUAqtMZihJnfS3mwecVJZiMTtzC1\r
  34         tt0g==\r
  35 Received: by 10.236.140.67 with SMTP id d43mr12026684yhj.19.1343890284856;\r
  36         Wed, 01 Aug 2012 23:51:24 -0700 (PDT)\r
  37 Received: from localhost ([2001:4b98:dc0:43:216:3eff:fe1b:25f3])\r
  38         by mx.google.com with ESMTPS id a4sm9565811yhj.11.2012.08.01.23.51.22\r
  39         (version=SSLv3 cipher=OTHER); Wed, 01 Aug 2012 23:51:24 -0700 (PDT)\r
  40 From: Jani Nikula <jani@nikula.org>\r
  41 To: Austin Clements <amdragon@MIT.EDU>, notmuch@notmuchmail.org\r
  42 Subject: Re: [PATCH v4 7/8] emacs: Fix navigation of multi-line search result\r
  43         formats\r
  44 In-Reply-To: <1342892232-5659-8-git-send-email-amdragon@mit.edu>\r
  45 References: <1341354059-29396-1-git-send-email-amdragon@mit.edu>\r
  46         <1342892232-5659-1-git-send-email-amdragon@mit.edu>\r
  47         <1342892232-5659-8-git-send-email-amdragon@mit.edu>\r
  48 User-Agent: Notmuch/0.13.2+106~gb810aee (http://notmuchmail.org) Emacs/23.2.1\r
  49         (x86_64-pc-linux-gnu)\r
  50 Date: Thu, 02 Aug 2012 08:51:19 +0200\r
  51 Message-ID: <87mx2dais8.fsf@nikula.org>\r
  52 MIME-Version: 1.0\r
  53 Content-Type: text/plain; charset=us-ascii\r
  54 X-Gm-Message-State:\r
  55  ALoCoQlES6nznbAMo+8h8s2OvKDCHPAr9Hjo6MPNYiIHlvdYKZpk3dWD1r09ql2HlsP5aVjItZeW\r
  56 X-BeenThere: notmuch@notmuchmail.org\r
  57 X-Mailman-Version: 2.1.13\r
  58 Precedence: list\r
  59 List-Id: "Use and development of the notmuch mail system."\r
  60         <notmuch.notmuchmail.org>\r
  61 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  62         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  63 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  64 List-Post: <mailto:notmuch@notmuchmail.org>\r
  65 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  66 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  67         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  68 X-List-Received-Date: Thu, 02 Aug 2012 06:51:27 -0000\r
  69 \r
  70 On Sat, 21 Jul 2012, Austin Clements <amdragon@MIT.EDU> wrote:\r
  71 > At this point, the only remaining functions that don't support\r
  72 > multi-line search result formats are the thread navigation functions.\r
  73 > This patch fixes that by rewriting them in terms of\r
  74 > notmuch-search-result-{beginning,end}.\r
  75 >\r
  76 > This changes the behavior of notmuch-search-previous-thread slightly\r
  77 > so that if point isn't at the beginning of a result, it first moves\r
  78 > point to the beginning of the result.\r
  79 \r
  80 Hi Austin, bisecting suggests this patch (committed as 5d0883e) breaks\r
  81 notmuch-show-archive-thread-then-next and subsequently\r
  82 notmuch-show-advance-and-archive for me. When showing the last thread in\r
  83 the search results, notmuch-show-archive-thread-then-next used to exit\r
  84 to the end of the search buffer. Now it redisplays the current\r
  85 thread. When I hit SPC to cruise through mails, there's no indication\r
  86 that I've reached the end of search results, other than the same thread\r
  87 being displayed over and over.\r
  88 \r
  89 BR,\r
  90 Jani.\r
  91 \r
  92 \r
  93 > ---\r
  94 >  emacs/notmuch.el |   13 ++++++++++---\r
  95 >  1 file changed, 10 insertions(+), 3 deletions(-)\r
  96 >\r
  97 > diff --git a/emacs/notmuch.el b/emacs/notmuch.el\r
  98 > index d092528..fd1836f 100644\r
  99 > --- a/emacs/notmuch.el\r
 100 > +++ b/emacs/notmuch.el\r
 101 > @@ -293,18 +293,25 @@ For a mouse binding, return nil."\r
 102 >  (defun notmuch-search-next-thread ()\r
 103 >    "Select the next thread in the search results."\r
 104 >    (interactive)\r
 105 > -  (forward-line 1))\r
 106 > +  (when (notmuch-search-get-result (notmuch-search-result-end))\r
 107 > +    (goto-char (notmuch-search-result-end))))\r
 108 >  \r
 109 >  (defun notmuch-search-previous-thread ()\r
 110 >    "Select the previous thread in the search results."\r
 111 >    (interactive)\r
 112 > -  (forward-line -1))\r
 113 > +  (if (notmuch-search-get-result)\r
 114 > +      (unless (bobp)\r
 115 > +     (goto-char (notmuch-search-result-beginning (- (point) 1))))\r
 116 > +    ;; We must be past the end; jump to the last result\r
 117 > +    (notmuch-search-last-thread)))\r
 118 >  \r
 119 >  (defun notmuch-search-last-thread ()\r
 120 >    "Select the last thread in the search results."\r
 121 >    (interactive)\r
 122 >    (goto-char (point-max))\r
 123 > -  (forward-line -2))\r
 124 > +  (forward-line -2)\r
 125 > +  (let ((beg (notmuch-search-result-beginning)))\r
 126 > +    (when beg (goto-char beg))))\r
 127 >  \r
 128 >  (defun notmuch-search-first-thread ()\r
 129 >    "Select the first thread in the search results."\r
 130 > -- \r
 131 > 1.7.10\r
 132 >\r
 133 > _______________________________________________\r
 134 > notmuch mailing list\r
 135 > notmuch@notmuchmail.org\r
 136 > http://notmuchmail.org/mailman/listinfo/notmuch\r