16/ff4f4d0485b65f8e4453fa6f398d655fb790b2

   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 B2998421181\r
   6         for <notmuch@notmuchmail.org>; Tue, 17 Jan 2012 08:18:25 -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 KciTxvECf5el for <notmuch@notmuchmail.org>;\r
  16         Tue, 17 Jan 2012 08:18:25 -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 DE791429E3B\r
  21         for <notmuch@notmuchmail.org>; Tue, 17 Jan 2012 08:18:24 -0800 (PST)\r
  22 Received: by bkty12 with SMTP id y12so707577bkt.26\r
  23         for <notmuch@notmuchmail.org>; Tue, 17 Jan 2012 08:18:22 -0800 (PST)\r
  24 MIME-Version: 1.0\r
  25 Received: by 10.205.120.148 with SMTP id fy20mr6950189bkc.125.1326817101973;\r
  26         Tue, 17 Jan 2012 08:18:21 -0800 (PST)\r
  27 Sender: awg@xvx.ca\r
  28 Received: by 10.205.33.73 with HTTP; Tue, 17 Jan 2012 08:18:21 -0800 (PST)\r
  29 X-Originating-IP: [208.77.198.202]\r
  30 In-Reply-To: <cuny5t6r97c.fsf@hotblack-desiato.hh.sledj.net>\r
  31 References: <1326737603-21166-1-git-send-email-awg+notmuch@xvx.ca>\r
  32         <1326737603-21166-5-git-send-email-awg+notmuch@xvx.ca>\r
  33         <cuny5t6r97c.fsf@hotblack-desiato.hh.sledj.net>\r
  34 Date: Tue, 17 Jan 2012 09:18:21 -0700\r
  35 X-Google-Sender-Auth: MzyqVCB__iSQtPOnhdUyF1ZlOCg\r
  36 Message-ID:\r
  37  <CAMoJFUs9++xK15U3xpmKRGLR6pi_vWTq76KFywW2oksopegrew@mail.gmail.com>\r
  38 Subject: Re: [PATCH v2 4/4] emacs: Use the new JSON reply format.\r
  39 From: Adam Wolfe Gordon <awg+notmuch@xvx.ca>\r
  40 To: David Edmondson <dme@dme.org>\r
  41 Content-Type: text/plain; charset=ISO-8859-1\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: Tue, 17 Jan 2012 16:18:25 -0000\r
  57 \r
  58 Hi David,\r
  59 \r
  60 Thanks for the review. A couple of comments inline:\r
  61 \r
  62 On Tue, Jan 17, 2012 at 02:04, David Edmondson <dme@dme.org> wrote:\r
  63 >> + =A0 =A0(insert "\n")\r
  64 >> + =A0 =A0(set-buffer-modified-p nil)))\r
  65 >\r
  66 > Is this newline always required? Is it the cause of the spurious blank\r
  67 > line down below?\r
  68 \r
  69 This is the cause of the spurious blank line, but without it the tests\r
  70 complain about missing a blank line at the end of the file.  There may\r
  71 be a better way to deal with this - I'll experiment.\r
  72 \r
  73 >> =A0(defun notmuch-mua-reply (query-string &optional sender reply-all)\r
  74 >> ...\r
  75 >> + =A0 =A0 =A0(insert (format "On %s, %s wrote:\n"\r
  76 >> + =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 (cdr (assq 'date original-headers)=\r
  77 )\r
  78 >> + =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 (cdr (assq 'from original-headers)=\r
  79 )))\r
  80 >\r
  81 > I wonder whether emacs should be regenerating this or not. I'm okay with\r
  82 > it, but previous discussion was that it should remain the responsibility\r
  83 > of the CLI.\r
  84 \r
  85 I like this being generated in the MUA because then it can be\r
  86 customized easily (e.g. in my later patch).  Of course, it would also\r
  87 be possible to add this as a config option for the CLI and generate it\r
  88 there, but it feels to me like if there's a line between notmuch and\r
  89 the MUA, this belongs on the MUA side, especially in MUAs that are\r
  90 formatting the reply themselves anyway.\r
  91 \r
  92 I'm happy to hear more discussion on this, and will implement whatever\r
  93 seems best.  I don't actually use the customization myself, it just\r
  94 strikes me as a useful feature.\r