33/d182d0ef099744fb0bacf1f99db888bff0d1c3

   1 Return-Path: <jrollins@finestructure.net>\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 A1F26429E5F\r
   6         for <notmuch@notmuchmail.org>; Wed, 18 Jan 2012 00:47:23 -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: -2.29\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-2.29 tagged_above=-999 required=5\r
  12         tests=[RCVD_IN_DNSWL_MED=-2.3, T_MIME_NO_TEXT=0.01] 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 ld48wdOaInhS for <notmuch@notmuchmail.org>;\r
  16         Wed, 18 Jan 2012 00:47:23 -0800 (PST)\r
  17 Received: from outgoing-mail.its.caltech.edu (outgoing-mail.its.caltech.edu\r
  18         [131.215.239.19])\r
  19         by olra.theworths.org (Postfix) with ESMTP id 15ECD429E27\r
  20         for <notmuch@notmuchmail.org>; Wed, 18 Jan 2012 00:47:23 -0800 (PST)\r
  21 Received: from fire-doxen.imss.caltech.edu (localhost [127.0.0.1])\r
  22         by fire-doxen-postvirus (Postfix) with ESMTP id 6BC2E2E50D11;\r
  23         Wed, 18 Jan 2012 00:47:20 -0800 (PST)\r
  24 X-Spam-Scanned: at Caltech-IMSS on fire-doxen by amavisd-new\r
  25 Received: from finestructure.net (cpe-76-174-137-84.socal.res.rr.com\r
  26         [76.174.137.84]) (Authenticated sender: jrollins)\r
  27         by fire-doxen-submit (Postfix) with ESMTP id A50372E506BF;\r
  28         Wed, 18 Jan 2012 00:47:14 -0800 (PST)\r
  29 Received: by finestructure.net (Postfix, from userid 1000)\r
  30         id 334FB262; Wed, 18 Jan 2012 00:47:14 -0800 (PST)\r
  31 From: Jameson Graef Rollins <jrollins@finestructure.net>\r
  32 To: David Edmondson <dme@dme.org>, Notmuch Mail <notmuch@notmuchmail.org>\r
  33 Subject: Re: [PATCH 4/6] emacs: add option to notmuch-show-next-open-message\r
  34         to pop out to parent buffer if at end\r
  35 In-Reply-To: <cunfwfdo2dw.fsf@hotblack-desiato.hh.sledj.net>\r
  36 References: <871uqy19yo.fsf@servo.finestructure.net>\r
  37         <1326823531-14549-1-git-send-email-jrollins@finestructure.net>\r
  38         <1326823531-14549-2-git-send-email-jrollins@finestructure.net>\r
  39         <1326823531-14549-3-git-send-email-jrollins@finestructure.net>\r
  40         <1326823531-14549-4-git-send-email-jrollins@finestructure.net>\r
  41         <cunfwfdo2dw.fsf@hotblack-desiato.hh.sledj.net>\r
  42 User-Agent: Notmuch/0.11+63~g6965a07 (http://notmuchmail.org) Emacs/23.3.1\r
  43         (x86_64-pc-linux-gnu)\r
  44 Date: Wed, 18 Jan 2012 00:47:11 -0800\r
  45 Message-ID: <87aa5lqtww.fsf@servo.finestructure.net>\r
  46 MIME-Version: 1.0\r
  47 Content-Type: multipart/signed; boundary="=-=-=";\r
  48         micalg=pgp-sha256; protocol="application/pgp-signature"\r
  49 X-BeenThere: notmuch@notmuchmail.org\r
  50 X-Mailman-Version: 2.1.13\r
  51 Precedence: list\r
  52 List-Id: "Use and development of the notmuch mail system."\r
  53         <notmuch.notmuchmail.org>\r
  54 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  55         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  56 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  57 List-Post: <mailto:notmuch@notmuchmail.org>\r
  58 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  59 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  60         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  61 X-List-Received-Date: Wed, 18 Jan 2012 08:47:23 -0000\r
  62 \r
  63 --=-=-=\r
  64 Content-Transfer-Encoding: quoted-printable\r
  65 \r
  66 On Wed, 18 Jan 2012 08:12:27 +0000, David Edmondson <dme@dme.org> wrote:\r
  67 > No need for brackets around `r'. Please put initialised local variables\r
  68 > before uninitialised.\r
  69 \r
  70 Yeah, that's another comment of Aron's that I forgot to fix this time\r
  71 around.  Sorry about that.\r
  72 \r
  73 > >      (while (and (setq r (notmuch-show-goto-message-next))\r
  74 > >             (not (notmuch-show-message-visible-p))))\r
  75 > >      (if r\r
  76 > >     (progn\r
  77 > >       (notmuch-show-mark-read)\r
  78 > >       (notmuch-show-message-adjust))\r
  79 > > -      (goto-char (point-max)))))\r
  80 > > +      (if (and parent-buffer pop-at-end)\r
  81 > > +     (progn\r
  82 > > +       (kill-this-buffer)\r
  83 > > +       (switch-to-buffer parent-buffer)\r
  84 > > +       (forward-line 1))\r
  85 > > +   (goto-char (point-max))))))\r
  86 >=20\r
  87 > Can you explain in words how this is expected to behave please?\r
  88 \r
  89 If there is not another message, but there is a parent buffer and the\r
  90 pop-at-end variable is set, kill this buffer, go the parent, and move to\r
  91 the next thread.  Otherwise, go to the max point in the buffer.\r
  92 \r
  93 Do you see a problem?\r
  94 \r
  95 The one thing I do realize now is that the behavior might be slightly\r
  96 strange if the parent buffer is itself a show buffer.  In that case, the\r
  97 forward-line part doesn't make any sense.  It should check that the\r
  98 parent is a search buffer first.  It should also use the notmuch-search\r
  99 function to jump to the next thread (notmuch-search-next-thread).  The\r
 100 notmuch-show-next-thread function could use the same improvements.\r
 101 \r
 102 jamie.\r
 103 \r
 104 --=-=-=\r
 105 Content-Type: application/pgp-signature\r
 106 \r
 107 -----BEGIN PGP SIGNATURE-----\r
 108 Version: GnuPG v1.4.11 (GNU/Linux)\r
 109 \r
 110 iQIcBAEBCAAGBQJPFocPAAoJEO00zqvie6q8lrsP/2UO2z+RclxA/p8ZB0N8ZOh4\r
 111 B7Suov6NAtTyDiIGSGxROhtZn97hpXMAW3rdFMnIIYLrvMH5vMbjDbLxJm1iozTq\r
 112 2rzE9izFi9TxdHEB7wrxd7Yd3dX/WNSy6IJIAvx2mYrYsC9ONBFXRLDS18AqZwCz\r
 113 8kq/EkxREMrvZkLvC+eLQ8yTkSJf8NEgwQlxdOwx78lOD0w7p0zntd8/EzOU9Df7\r
 114 jA6Sjb/GQELqm4x7+MVxorrXTFbqv2SMS1wZitaVSNA1ewXTLP9FxwnHowi/4KW3\r
 115 L0Xci315+DqI4VzkIdaIE+LdlUevWxMxU3wmkNZTlNwO596QU4KeFUQ11/R55UAd\r
 116 T3bcvfqTDbFZ/U3Opbr46F6LztNzn3+y7rjzE1tIg14KArNcU7I6WffXPKXpBAh8\r
 117 UoLYOkJmhJlx7AAdo5z69vdgv7LFzGFkoEgTylhoz+mRLPVPwD5taZuvCYPbbuHd\r
 118 kyJW6nN+j4nMZOExDBrjdO7xMWsPXREbk6bHzEaGrxbDAJOJQ27Eoa4g9OWM/ZAp\r
 119 VMdHZVoePh8iVWhlfHwImsjMprCNralGaVIlaA2JhhWqq4upts73Nj0PCG9oCzBf\r
 120 95I5MlcyjTUtH7vebEuw83OyK0jP7VRBfgBBMUcKIpsa7E4u50Z9u0XQdUBiaqAG\r
 121 btqehWD7nvP+AXSUJW1q\r
 122 =q4eB\r
 123 -----END PGP SIGNATURE-----\r
 124 --=-=-=--\r