86/d30631c2bf8d840e729ef40d566d4824a84f9c

   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 67CDB431FBD;\r
   6         Fri, 11 Dec 2009 16:02:08 -0800 (PST)\r
   7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
   8 Received: from olra.theworths.org ([127.0.0.1])\r
   9         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  10         with ESMTP id UXUIzxDR5r3l; Fri, 11 Dec 2009 16:02:07 -0800 (PST)\r
  11 Received: from yoom.home.cworth.org (localhost [127.0.0.1])\r
  12         by olra.theworths.org (Postfix) with ESMTP id 770D3431FAE;\r
  13         Fri, 11 Dec 2009 16:02:07 -0800 (PST)\r
  14 Received: by yoom.home.cworth.org (Postfix, from userid 1000)\r
  15         id 31C7E254306; Fri, 11 Dec 2009 16:02:07 -0800 (PST)\r
  16 From: Carl Worth <cworth@cworth.org>\r
  17 To: Dirk Hohndel <hohndel@infradead.org>, notmuch@notmuchmail.org\r
  18 In-Reply-To: <87y6l92yuw.fsf@yoom.home.cworth.org>\r
  19 References: <1260511813.3341.22.camel@dhohndel-mobl.amr.corp.intel.com>\r
  20         <874onx4gyl.fsf@yoom.home.cworth.org>\r
  21         <87y6l92yuw.fsf@yoom.home.cworth.org>\r
  22 Date: Fri, 11 Dec 2009 16:02:06 -0800\r
  23 Message-ID: <87r5r12h6p.fsf@yoom.home.cworth.org>\r
  24 MIME-Version: 1.0\r
  25 Content-Type: multipart/signed; boundary="=-=-=";\r
  26         micalg=pgp-sha1; protocol="application/pgp-signature"\r
  27 Subject: Re: [notmuch] emacs mode performance issue\r
  28 X-BeenThere: notmuch@notmuchmail.org\r
  29 X-Mailman-Version: 2.1.12\r
  30 Precedence: list\r
  31 List-Id: "Use and development of the notmuch mail system."\r
  32         <notmuch.notmuchmail.org>\r
  33 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  34         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  35 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  36 List-Post: <mailto:notmuch@notmuchmail.org>\r
  37 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  38 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  39         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  40 X-List-Received-Date: Sat, 12 Dec 2009 00:02:08 -0000\r
  41 \r
  42 --=-=-=\r
  43 \r
  44 On Fri, 11 Dec 2009 09:40:23 -0800, Carl Worth <cworth@cworth.org> wrote:\r
  45 > Which should let you tar these up and you can send them to me privately\r
  46 > and I'll try to replicate and fix the bug.\r
  47 \r
  48 Thanks for passing on the messages, Dirk. If anyone's curious the\r
  49 message that triggered the bug is a public mail on the Linux Kernel\r
  50 Mailing List:\r
  51 \r
  52         id:alpine.LFD.2.00.0912081304070.3560@localhost.localdomain\r
  53 \r
  54 (It does have to be viewed in thread context---indented at least 13\r
  55 columns to trigger the bug.)\r
  56 \r
  57 The bug was an infinite loop add the button for a hidden citation,\r
  58 (where our emacs lisp code got a bit confused about where the next line\r
  59 was and infinitely looped on the same citation adding a never ending\r
  60 sequence of "[1-line citation]" buttons).\r
  61 \r
  62 I fixed the infinite loop and while in the area made a few other minor\r
  63 improvements to the citation-hiding code:\r
  64 \r
  65   * No longer consider '>' as introducing a citation if not in first\r
  66     column.\r
  67 \r
  68   * No longer insert an the extra blank line after the citation button.\r
  69 \r
  70   * Add text to the citation button to tell new users that they can\r
  71     click or press Enter on the button to show the citation.\r
  72 \r
  73 I'd still like to add one more feature which is a keybinding to make all\r
  74 the citations for the current message visible. (It could even do the\r
  75 whole thread like the button we used to have, but *only* if it ensured\r
  76 that the current position of point within the current buffer and the\r
  77 current window remained unchanged.)\r
  78 \r
  79 I hope that's helpful,\r
  80 \r
  81 -Carl\r
  82 \r
  83 --=-=-=\r
  84 Content-Type: application/pgp-signature\r
  85 \r
  86 -----BEGIN PGP SIGNATURE-----\r
  87 Version: GnuPG v1.4.10 (GNU/Linux)\r
  88 \r
  89 iD8DBQFLIt1/6JDdNq8qSWgRAj93AKCRoyVUbcAfpEHEE1zuVZzxJCqnEwCfSa1d\r
  90 vVX1WZmd1aAS99L/Nh/+FP0=\r
  91 =GYah\r
  92 -----END PGP SIGNATURE-----\r
  93 --=-=-=--\r