c0/2bd64172042d2d5d4822f8ee01c3dad32cf356

   1 Return-Path: <awg@xvx.ca>\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 C108F431FAF\r
   6         for <notmuch@notmuchmail.org>; Thu, 19 Jan 2012 20:46:50 -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: -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 fu4yS8lgY1LP for <notmuch@notmuchmail.org>;\r
  16         Thu, 19 Jan 2012 20:46:50 -0800 (PST)\r
  17 Received: from mail-bk0-f53.google.com (mail-bk0-f53.google.com\r
  18         [209.85.214.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 DE025431FAE\r
  21         for <notmuch@notmuchmail.org>; Thu, 19 Jan 2012 20:46:49 -0800 (PST)\r
  22 Received: by bkbzu17 with SMTP id zu17so154689bkb.26\r
  23         for <notmuch@notmuchmail.org>; Thu, 19 Jan 2012 20:46:47 -0800 (PST)\r
  24 MIME-Version: 1.0\r
  25 Received: by 10.205.131.142 with SMTP id hq14mr7657296bkc.89.1327034806913;\r
  26         Thu, 19 Jan 2012 20:46:46 -0800 (PST)\r
  27 Sender: awg@xvx.ca\r
  28 Received: by 10.204.49.75 with HTTP; Thu, 19 Jan 2012 20:46:46 -0800 (PST)\r
  29 X-Originating-IP: [96.52.216.56]\r
  30 In-Reply-To: <m2k44n4jlz.fsf@wal122.wireless-pennnet.upenn.edu>\r
  31 References: <1326995217-27423-1-git-send-email-awg+notmuch@xvx.ca>\r
  32         <1326995217-27423-6-git-send-email-awg+notmuch@xvx.ca>\r
  33         <m2k44n4jlz.fsf@wal122.wireless-pennnet.upenn.edu>\r
  34 Date: Thu, 19 Jan 2012 21:46:46 -0700\r
  35 X-Google-Sender-Auth: 7sumT2DAuv3zly9bD9TNE931GvA\r
  36 Message-ID:\r
  37  <CAMoJFUtn2Ls4maJGecY5Y8HzdOJSR6ZfVdUxnCZ6s4Jhq8r9yw@mail.gmail.com>\r
  38 Subject: Re: [PATCH v3 5/5] emacs: Use message-citation-line-format in reply\r
  39 From: Adam Wolfe Gordon <awg+notmuch@xvx.ca>\r
  40 To: Aaron Ecay <ecay@sas.upenn.edu>\r
  41 Content-Type: text/plain; charset=windows-1252\r
  42 Content-Transfer-Encoding: quoted-printable\r
  43 Cc: notmuch@notmuchmail.org\r
  44 X-BeenThere: notmuch@notmuchmail.org\r
  45 X-Mailman-Version: 2.1.13\r
  46 Precedence: list\r
  47 List-Id: "Use and development of the notmuch mail system."\r
  48         <notmuch.notmuchmail.org>\r
  49 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  50         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  51 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  52 List-Post: <mailto:notmuch@notmuchmail.org>\r
  53 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  54 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  55         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  56 X-List-Received-Date: Fri, 20 Jan 2012 04:46:50 -0000\r
  57 \r
  58 On Thu, Jan 19, 2012 at 11:45, Aaron Ecay <ecay@sas.upenn.edu> wrote:\r
  59 > Shouldn=92t this just use message-insert-formatted-citation-line?\r
  60 \r
  61 Yes, good idea.  I just tried this and it almost works.  The only\r
  62 issue is that the default message-mode-citation-line-format has a\r
  63 newline at the end, and this function inserts an *additional* newline,\r
  64 so we end up with a blank line before the beginning of the quoted\r
  65 text.  This is fixable by the user, of course, but it means the\r
  66 default out-of-the-box setup will create funny-looking replies, which\r
  67 is probably bad.  Thoughts?\r
  68 \r
  69 > Another approach you might take with this patch series is to look at\r
  70 > the message-cite-original function (which I just discovered as I was\r
  71 > plumbing around in message.el looking for the function to format the\r
  72 > citation line). =A0I think that what one does to use this fn is to put\r
  73 > the original message text into the reply buffer (unquoted), set point\r
  74 > and mark to encompass it, then call the fn. =A0It automatically handles\r
  75 > inserting the quotes, and has some customization options (stripping\r
  76 > signatures from replies, customizable quote character instead of =93>=94,\r
  77 > ...).\r
  78 >\r
  79 > The message-cite-original function also adds escape characters to the\r
  80 > cookies that message-mode uses to indicate sign/encrypt/attach\r
  81 > directives. =A0I think notmuch exposes files on the user=92s computer to\r
  82 > others, if a user can be tricked into replying to a message with an\r
  83 > attachment cookie and not stripping the cookie from the reply text. =A0So\r
  84 > to mitigate this, whatever reply mechanism winds up being used should\r
  85 > call mml-quote-region on the reply text (as message-cite-original does).\r
  86 \r
  87 I've also tried using message-cite-original to create the reply body,\r
  88 and it also almost works.  The issue, again, is one of defaults.  In\r
  89 addition to the blank line I mentioned above, the default\r
  90 message-citation-line-function inserts the "plain" citation line "Foo\r
  91 <foo@bar.baz> writes:" instead of the formatted one.  This is a big\r
  92 change from the current notmuch default.\r
  93 \r
  94 If everyone's OK with this and willing to customize it themselves,\r
  95 then I'm happy to go with this solution, but I'm pretty reluctant to\r
  96 change the default in such a significant way.\r