Re: [PATCH v5.2 7/7] emacs: Use the new JSON reply format and message-cite-original

diff --git a/b5/e115b9f65515dc642bf31e5a104c66e4b06326 b/b5/e115b9f65515dc642bf31e5a104c66e4b06326
new file mode 100644 (file)
index 0000000..73a1591
--- /dev/null
+++ b/b5/e115b9f65515dc642bf31e5a104c66e4b06326
@@ -0,0 +1,101 @@
+Return-Path: <awg@xvx.ca>\r
+X-Original-To: notmuch@notmuchmail.org\r
+Delivered-To: notmuch@notmuchmail.org\r
+Received: from localhost (localhost [127.0.0.1])\r
+       by olra.theworths.org (Postfix) with ESMTP id 1D884431FB6\r
+       for <notmuch@notmuchmail.org>; Tue, 21 Feb 2012 22:46:17 -0800 (PST)\r
+X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
+X-Spam-Flag: NO\r
+X-Spam-Score: -0.7\r
+X-Spam-Level: \r
+X-Spam-Status: No, score=-0.7 tagged_above=-999 required=5\r
+       tests=[RCVD_IN_DNSWL_LOW=-0.7] autolearn=disabled\r
+Received: from olra.theworths.org ([127.0.0.1])\r
+       by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
+       with ESMTP id GlJc2Hj+4oIi for <notmuch@notmuchmail.org>;\r
+       Tue, 21 Feb 2012 22:46:16 -0800 (PST)\r
+Received: from mail-lpp01m010-f53.google.com (mail-lpp01m010-f53.google.com\r
+       [209.85.215.53]) (using TLSv1 with cipher RC4-SHA (128/128 bits))\r
+       (No client certificate requested)\r
+       by olra.theworths.org (Postfix) with ESMTPS id 398D8431FAE\r
+       for <notmuch@notmuchmail.org>; Tue, 21 Feb 2012 22:46:15 -0800 (PST)\r
+Received: by lahd3 with SMTP id d3so9000294lah.26\r
+       for <notmuch@notmuchmail.org>; Tue, 21 Feb 2012 22:46:12 -0800 (PST)\r
+Received-SPF: pass (google.com: domain of awg@xvx.ca designates 10.152.102.237\r
+       as permitted sender) client-ip=10.152.102.237; \r
+Authentication-Results: mr.google.com;\r
+       spf=pass (google.com: domain of awg@xvx.ca designates\r
+       10.152.102.237 as permitted sender) smtp.mail=awg@xvx.ca\r
+Received: from mr.google.com ([10.152.102.237])\r
+       by 10.152.102.237 with SMTP id fr13mr21997935lab.10.1329893172980\r
+       (num_hops = 1); Tue, 21 Feb 2012 22:46:12 -0800 (PST)\r
+MIME-Version: 1.0\r
+Received: by 10.152.102.237 with SMTP id fr13mr18424909lab.10.1329893172915;\r
+       Tue, 21 Feb 2012 22:46:12 -0800 (PST)\r
+Sender: awg@xvx.ca\r
+Received: by 10.112.114.103 with HTTP; Tue, 21 Feb 2012 22:46:12 -0800 (PST)\r
+X-Originating-IP: [96.52.216.56]\r
+In-Reply-To: <20120217200017.GG5991@mit.edu>\r
+References: <1329361957-28493-1-git-send-email-awg+notmuch@xvx.ca>\r
+       <1329361957-28493-8-git-send-email-awg+notmuch@xvx.ca>\r
+       <20120217200017.GG5991@mit.edu>\r
+Date: Tue, 21 Feb 2012 23:46:12 -0700\r
+X-Google-Sender-Auth: WPQOsapDmpr1XIm2hqmD_xAGHBM\r
+Message-ID:\r
+ <CAMoJFUszH+RjfOj2qpFL6ZYpG418ZtW9za56pzD7gxhp70BJFw@mail.gmail.com>\r
+Subject: Re: [PATCH v5.2 7/7] emacs: Use the new JSON reply format and\r
+       message-cite-original\r
+From: Adam Wolfe Gordon <awg+notmuch@xvx.ca>\r
+To: Austin Clements <amdragon@mit.edu>\r
+Content-Type: text/plain; charset=ISO-8859-1\r
+Content-Transfer-Encoding: quoted-printable\r
+X-Gm-Message-State:\r
+ ALoCoQlDjRefGRt6cLtC1ou4vELXCH9Uo7r45nhITIupi2PcZGBZdPe/IxnqIOphc+lai20B5H4a\r
+Cc: notmuch@notmuchmail.org\r
+X-BeenThere: notmuch@notmuchmail.org\r
+X-Mailman-Version: 2.1.13\r
+Precedence: list\r
+List-Id: "Use and development of the notmuch mail system."\r
+       <notmuch.notmuchmail.org>\r
+List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
+       <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
+List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
+List-Post: <mailto:notmuch@notmuchmail.org>\r
+List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
+List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
+       <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
+X-List-Received-Date: Wed, 22 Feb 2012 06:46:17 -0000\r
+\r
+Hi Austin,\r
+\r
+A couple of quick notes below, just before I send the new version:\r
+\r
+On Fri, Feb 17, 2012 at 13:00, Austin Clements <amdragon@mit.edu> wrote:\r
+> One of the biggest wins of using consistent JSON parsing settings is\r
+> that this can be replaced with notmuch-show-mm-display-part-inline,\r
+> which, as far as I can tell, accomplishes the same thing, but handles\r
+> a lot of corner-cases that this doesn't (like crypto and charset\r
+> conversion).\r
+\r
+This would be very desirable indeed, but after taking a stab at it,\r
+I've discovered it's a really significant amount of work. The show\r
+code relies on a bunch of buffer-local variables regarding crypto and\r
+such, which are tricky to factor out into notmuch-lib.\r
+\r
+Do you think it's acceptable to leave this as future work?\r
+\r
+> Alternatively, notmuch-mua-get-quotable-parts could be\r
+> notmuch-mua-insert-quotable-parts, which would probably simplify\r
+> things a bit. =A0Your call.\r
+\r
+I've decided to leave this as get-quotable-parts. This is mostly my\r
+personal preference to have a single function do the insertions when\r
+possible, which I think makes the flow more obvious.\r
+\r
+> Is message-cite-original guaranteed to leave point in a reasonable\r
+> place for this or should we create our own marker above (probably\r
+> after the if re-search-backward..) and use it here to get point to the\r
+> right place?\r
+\r
+It turns out it actually wasn't, which is why MML tags weren't getting\r
+escaped twice. I've fixed this.\r