a5/2b547948328d160f89e82d0b53bbdb1d67c038

   1 Return-Path: <dme@dme.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 1C1CE431FBD\r
   6         for <notmuch@notmuchmail.org>; Tue, 13 May 2014 03:02:15 -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 s+LVSQHiwBMh for <notmuch@notmuchmail.org>;\r
  16         Tue, 13 May 2014 03:02:07 -0700 (PDT)\r
  17 Received: from mail-we0-f169.google.com (mail-we0-f169.google.com\r
  18         [74.125.82.169]) (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 39E19431FAF\r
  21         for <notmuch@notmuchmail.org>; Tue, 13 May 2014 03:02:07 -0700 (PDT)\r
  22 Received: by mail-we0-f169.google.com with SMTP id u56so119807wes.28\r
  23         for <notmuch@notmuchmail.org>; Tue, 13 May 2014 03:02:06 -0700 (PDT)\r
  24 X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;\r
  25         d=1e100.net; s=20130820;\r
  26         h=x-gm-message-state:to:subject:in-reply-to:references:user-agent\r
  27         :from:date:message-id:mime-version:content-type;\r
  28         bh=3DThrqgYVORoc4xFdqPog6dPgYfLOJewrgMnGXddAUo=;\r
  29         b=Hdz3YMnugQdkJWid4gDTKz//pIs7bfuIjPm2c9+69TeQ24Wy1zUimnxAOSwc7X/FQV\r
  30         mYeGfdZECgvCDCdhPojNw/zZTp+0cCOgEB1Xj76eEmZ7Dc0wgUzKDxc7tfPHiRODXe9v\r
  31         eOjriNMXFs2kK4YqDqMWHf4h0M+kmU3DDOJmwoDpD/hpoRSrtYdhgn7akOLoyjCeNUgA\r
  32         fL2Gd7UB29913c9rJaBKfD2uSZ895MkP+FVSmz9fnQqxT25Gcly+vQuYEv9oo67ZRhfG\r
  33         OaeEvH/nUEjTDZx6icH5yPkyzEljGxl13rcNgOfHMY9IPKogl5I2q4ydbDJN3XLWgPlh\r
  34         eEYQ==\r
  35 X-Gm-Message-State:\r
  36  ALoCoQnGy1YTqKSAb1nHsNHODpOvB+shFxpNTJ9n/nJm9jEXdR8QZbvpiq2mceT+JzMXdSFjqhsA\r
  37 X-Received: by 10.180.99.40 with SMTP id en8mr20470196wib.24.1399975324912;\r
  38         Tue, 13 May 2014 03:02:04 -0700 (PDT)\r
  39 Received: from localhost ([2a01:348:1a2:1:a288:b4ff:fe8a:77d8])\r
  40         by mx.google.com with ESMTPSA id\r
  41         mw4sm21035587wib.12.2014.05.13.03.02.02 for <multiple recipients>\r
  42         (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128);\r
  43         Tue, 13 May 2014 03:02:03 -0700 (PDT)\r
  44 To: Mark Walters <markwalters1009@gmail.com>, notmuch@notmuchmail.org\r
  45 Subject: Re: [PATCH v3 0/9] emacs: Improve the cited message included in\r
  46         replies\r
  47 In-Reply-To: <87vbtak5uz.fsf@qmul.ac.uk>\r
  48 References: <87sixdujkv.fsf@qmul.ac.uk>\r
  49         <1399897769-26809-1-git-send-email-dme@dme.org>\r
  50         <87vbtak5uz.fsf@qmul.ac.uk>\r
  51 User-Agent: Notmuch/0.18 (http://notmuchmail.org) Emacs/24.3.1\r
  52         (x86_64-pc-linux-gnu)\r
  53 From: David Edmondson <dme@dme.org>\r
  54 Date: Tue, 13 May 2014 11:01:58 +0100\r
  55 Message-ID: <cunvbta5755.fsf@hotblack-desiato.hh.sledj.net>\r
  56 MIME-Version: 1.0\r
  57 Content-Type: multipart/signed; boundary="=-=-=";\r
  58         micalg=pgp-sha1; protocol="application/pgp-signature"\r
  59 X-BeenThere: notmuch@notmuchmail.org\r
  60 X-Mailman-Version: 2.1.13\r
  61 Precedence: list\r
  62 List-Id: "Use and development of the notmuch mail system."\r
  63         <notmuch.notmuchmail.org>\r
  64 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  65         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  66 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  67 List-Post: <mailto:notmuch@notmuchmail.org>\r
  68 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  69 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  70         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  71 X-List-Received-Date: Tue, 13 May 2014 10:02:15 -0000\r
  72 \r
  73 --=-=-=\r
  74 Content-Type: text/plain\r
  75 Content-Transfer-Encoding: quoted-printable\r
  76 \r
  77 Firstly, I don't think that the code resulting from this patch series is\r
  78 beyond improvement - the intention was really only that it be better\r
  79 than the current implementation.\r
  80 \r
  81 On Mon, May 12 2014, Mark Walters wrote:\r
  82 > On Mon, 12 May 2014, David Edmondson <dme@dme.org> wrote:\r
  83 >> emacs: Improve the cited message included in replies\r
  84 >>\r
  85 >> I tried to do things in small increments to make it easier to review.\r
  86 >>\r
  87 >> v2:\r
  88 >> - Don't run the text/plain hooks when generating the message to quote.\r
  89 >>\r
  90 >> v3:\r
  91 >> - Remove the 'no-button code, as it's no longer used.\r
  92 >> - Control the insertion of part headers using a function.\r
  93 >> - Fix the tests.\r
  94 >\r
  95 > I think I broadly like this series. I haven't thought through all the\r
  96 > ramifications yet so this is just some first thoughts. I will also send\r
  97 > some comments on individual patches.\r
  98 \r
  99 Thanks!\r
 100 \r
 101 > In notmuch-show we go to notmuch-show-insert-part-*/* to\r
 102 > notmuch-mm-display-part-inline and then leave the decision to inline to\r
 103 > mm-inlined-types. I think this means that, by default, we will not\r
 104 > inline signatures amongst other things.\r
 105 \r
 106 The rule is essentially: whatever text would be shown when the message\r
 107 is displayed for reading, without any of the washing.\r
 108 \r
 109 > So at the least I think we should decide whether we want to override\r
 110 > mm-inlined-types.\r
 111 \r
 112 I'm not really clear on the benefits of this. Could you explain?\r
 113 \r
 114 > Alternatively, and in my view preferably, we could have a function or\r
 115 > variable of our own which says which parts to include. Indeed, if do\r
 116 > it with a function we might be able to make an option to reply to mean\r
 117 > "include parts currently shown in the notmuch-show buffer" which might\r
 118 > be nice.\r
 119 \r
 120 That seems over complicated to me. The rule (above) from this series\r
 121 is easy to understand and work with. Other mechanisms could be\r
 122 implemented later, of course.\r
 123 \r
 124 > There is a related question and possible bug that we might be able to\r
 125 > do something about at the same time: should we include text parts in the\r
 126 > reply if they have content-disposition attachment? I have been caught\r
 127 > about by this on one occasion replying to a message with a 500K log file\r
 128 > attached (and notmuch-show/wash becomes very slow with a 500K\r
 129 > message!)\r
 130 \r
 131 This is really a question of what happens in `show' mode. If it is\r
 132 currently displaying text parts with content-disposition attachment,\r
 133 that sounds like a bug that should be fixed (which would mean that the\r
 134 cited message wouldn't include that part either).\r
 135 \r
 136 > Finally, I am not sure whether I like having buttons in the reply. My\r
 137 > instinct is against, but they do add context.\r
 138 \r
 139 The last patch in the series is an example of trying to do the right\r
 140 thing - show the part headers when they are necessary for proper\r
 141 understanding, but elide them in all other cases.\r
 142 \r
 143 The mechanism used to do this is pretty crude in the patch. One could\r
 144 imagine a better implementation that examines the depth of the part\r
 145 tree, etc.\r
 146 \r
 147 > Best wishes\r
 148 >\r
 149 > Mark\r
 150 >\r
 151 >\r
 152 >\r
 153 >\r
 154 >>\r
 155 >>\r
 156 >> David Edmondson (9):\r
 157 >>   emacs/show: Re-arrange determination if a part header is necessary\r
 158 >>   emacs/show: Allow the user to decide when part headers should be\r
 159 >>     inserted\r
 160 >>   emacs/show: Accommodate the lack of part header buttons\r
 161 >>   emacs/mua: Generate improved cited text for replies\r
 162 >>   emacs/show: Remove the 'no-buttons option of\r
 163 >>     `notmuch-show-insert-bodypart'\r
 164 >>   emacs/mua: Don't insert part headers in citations\r
 165 >>   test: Update the test output to accord with the reply changes\r
 166 >>   emacs/mua: Insert part headers depending on the message\r
 167 >>   test: Update the test output to accord with more reply changes\r
 168 >>\r
 169 >>  emacs/notmuch-mua.el  |  82 +++++++++++++++++++-----------\r
 170 >>  emacs/notmuch-show.el | 135 +++++++++++++++++++++++++++++++------------=\r
 171 -------\r
 172 >>  test/T310-emacs.sh    |  44 ++++++++++++++++\r
 173 >>  3 files changed, 180 insertions(+), 81 deletions(-)\r
 174 >>\r
 175 >> --=20\r
 176 >> 2.0.0.rc0\r
 177 >>\r
 178 >> _______________________________________________\r
 179 >> notmuch mailing list\r
 180 >> notmuch@notmuchmail.org\r
 181 >> http://notmuchmail.org/mailman/listinfo/notmuch\r
 182 \r
 183 --=-=-=\r
 184 Content-Type: application/pgp-signature; name="signature.asc"\r
 185 \r
 186 -----BEGIN PGP SIGNATURE-----\r
 187 Version: GnuPG v1\r
 188 \r
 189 iKYEARECAGYFAlNx7ZZfFIAAAAAALgAoaXNzdWVyLWZwckBub3RhdGlvbnMub3Bl\r
 190 bnBncC5maWZ0aGhvcnNlbWFuLm5ldDc1M0Y5NDJDMEExNjc3MDE4OURGMUYyMDY5\r
 191 RUNEMEFCRjA0OTY1MTYACgkQaezQq/BJZRYq8QCeOUu0kzUhaHypeKMf94pxt/7+\r
 192 aacAoIyc1cHTTjq2CoeLeIs8WUqH9ok7\r
 193 =Xbt6\r
 194 -----END PGP SIGNATURE-----\r
 195 --=-=-=--\r