87/dddb8e1dca53bcba7d140c1f6e940f566e1e33

   1 Return-Path: <joyfulgirl@archlinux.us>\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 C2681429E21\r
   6         for <notmuch@notmuchmail.org>; Fri, 28 Oct 2011 17:32:20 -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 Xlx9IOP-AVaY for <notmuch@notmuchmail.org>;\r
  16         Fri, 28 Oct 2011 17:32:20 -0700 (PDT)\r
  17 Received: from mail-qw0-f53.google.com (mail-qw0-f53.google.com\r
  18         [209.85.216.53]) (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 1F0AC431FB6\r
  21         for <notmuch@notmuchmail.org>; Fri, 28 Oct 2011 17:32:20 -0700 (PDT)\r
  22 Received: by qadc1 with SMTP id c1so5303933qad.26\r
  23         for <notmuch@notmuchmail.org>; Fri, 28 Oct 2011 17:32:19 -0700 (PDT)\r
  24 Received: by 10.224.177.68 with SMTP id bh4mr4873940qab.48.1319848339533;\r
  25         Fri, 28 Oct 2011 17:32:19 -0700 (PDT)\r
  26 Received: from localhost (209-6-79-248.c3-0.abr-ubr1.sbo-abr.ma.cable.rcn.com.\r
  27         [209.6.79.248])\r
  28         by mx.google.com with ESMTPS id do8sm17237075qab.17.2011.10.28.17.32.18\r
  29         (version=TLSv1/SSLv3 cipher=OTHER);\r
  30         Fri, 28 Oct 2011 17:32:19 -0700 (PDT)\r
  31 From: Ivy Foster <joyfulgirl@archlinux.us>\r
  32 To: notmuch@notmuchmail.org\r
  33 Subject: [Patch] Fix Emacs interface bug dealing with invisible text\r
  34 Date: Fri, 28 Oct 2011 20:29:49 -0400\r
  35 Message-ID: <878vo47i82.fsf@nausicaa.localdomain>\r
  36 MIME-Version: 1.0\r
  37 Content-Type: text/plain; charset=us-ascii\r
  38 X-BeenThere: notmuch@notmuchmail.org\r
  39 X-Mailman-Version: 2.1.13\r
  40 Precedence: list\r
  41 List-Id: "Use and development of the notmuch mail system."\r
  42         <notmuch.notmuchmail.org>\r
  43 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  44         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  45 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  46 List-Post: <mailto:notmuch@notmuchmail.org>\r
  47 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  48 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  49         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  50 X-List-Received-Date: Sat, 29 Oct 2011 00:32:20 -0000\r
  51 \r
  52 Hey, folks,\r
  53 \r
  54 I'm new to notmuch, but like much of what I've tried so far.\r
  55 Thanks to all the contributors!\r
  56 \r
  57 The following patch fixes a bug I've noticed in Emacs's\r
  58 notmuch-show mode. Specifically, if you:\r
  59 \r
  60     - have certain kinds of washing turned on (I've noticed\r
  61       it with signature hiding), and\r
  62 \r
  63     - are looking at the last or only message in a thread\r
  64 \r
  65 If you then hit Space (`notmuch-show-advance-or-archive') to\r
  66 move on, Emacs will fail to advance and/or archive the\r
  67 message, due to being kicked out of area of the invisible\r
  68 text before it can do what it's gotta do.\r
  69 \r
  70 The patch should fix the problem at the root, which seems to\r
  71 be with notmuch-show-move-past-invisible-[forward|backward].\r
  72 \r
  73 So far, I'm the only tester. However, I have added a\r
  74 signature to the end of this message for your convenience.\r
  75 (-:\r
  76 \r
  77 Share & Enjoy!\r
  78 \r
  79 Signed-off-by: Ivy Foster <joyfulgirl@archlinux.us>\r
  80 >From 61ac45b32b70f2e789004359e219bfe6b77c701e Mon Sep 17 00:00:00 2001\r
  81 From: Ivy Foster <joyfulgirl@archlinux.us>\r
  82 Date: Fri, 28 Oct 2011 19:30:45 -0400\r
  83 Subject: [PATCH 1/1] Move past invisible text properly in Emacs\r
  84 \r
  85     move-past-invisible-text-* in notmuch-show were\r
  86     interacting oddly with hidden text at the end of\r
  87     a thread, such as a washed signature block.\r
  88 ---\r
  89  emacs/notmuch-show.el |   14 +++++++++-----\r
  90  1 files changed, 9 insertions(+), 5 deletions(-)\r
  91 \r
  92 diff --git a/emacs/notmuch-show.el b/emacs/notmuch-show.el\r
  93 index 440e089..f14696b 100644\r
  94 --- a/emacs/notmuch-show.el\r
  95 +++ b/emacs/notmuch-show.el\r
  96 @@ -1017,12 +1017,16 @@ All currently available key bindings:\r
  97      t))\r
  98 \r
  99  (defun notmuch-show-move-past-invisible-forward ()\r
 100 -  (while (point-invisible-p)\r
 101 -    (forward-char)))\r
 102 +  (let ((disable-point-adjustment t))\r
 103 +    (goto-char (1+ (point)))\r
 104 +    (while (invisible-p (point))\r
 105 +      (goto-char (1+ (point))))))\r
 106 \r
 107  (defun notmuch-show-move-past-invisible-backward ()\r
 108 -  (while (point-invisible-p)\r
 109 -    (backward-char)))\r
 110 +  (let ((disable-point-adjustment t))\r
 111 +    (goto-char (1- (point)))\r
 112 +    (while (invisible-p (point))\r
 113 +      (goto-char (1- (point))))))\r
 114 \r
 115  ;; Functions relating to the visibility of messages and their\r
 116  ;; components.\r
 117 @@ -1160,7 +1164,7 @@ which this thread was originally shown."\r
 118       ;; go to the end, walk back over invisible text and then see if\r
 119       ;; point is visible.\r
 120       ((save-excursion\r
 121 -       (goto-char (- end-of-this-message 1))\r
 122 +       (goto-char end-of-this-message)\r
 123         (notmuch-show-move-past-invisible-backward)\r
 124         (> (point) (window-end)))\r
 125        ;; The bottom of this message is not visible - scroll.\r
 126 -- \r
 127 1.7.7.1\r
 128 \r
 129 P.S.: I also tried this with (point-invisible-p) instead of\r
 130       (invisible-p (point)), but for whatever reason, it\r
 131       didn't work. That may be something to look into as\r
 132       well.\r
 133 \r
 134 -- \r
 135 Quid me anxius sum?\r
 136 [ What? Me, worry? ]\r