Re: [PATCH] emacs: Reuse rather than reinvent message header filtering
authorDavid Edmondson <dme@dme.org>
Thu, 29 Apr 2010 09:00:34 +0000 (10:00 +0100)
committerW. Trevor King <wking@tremily.us>
Fri, 7 Nov 2014 17:37:00 +0000 (09:37 -0800)
61/70039fd2c33c442c433e01f3a63d04f8012505 [new file with mode: 0644]

diff --git a/61/70039fd2c33c442c433e01f3a63d04f8012505 b/61/70039fd2c33c442c433e01f3a63d04f8012505
new file mode 100644 (file)
index 0000000..90fcd04
--- /dev/null
@@ -0,0 +1,89 @@
+Return-Path: <dme@dme.org>\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 869764196F0\r
+       for <notmuch@notmuchmail.org>; Thu, 29 Apr 2010 02:00:24 -0700 (PDT)\r
+X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
+X-Spam-Flag: NO\r
+X-Spam-Score: -1.89\r
+X-Spam-Level: \r
+X-Spam-Status: No, score=-1.89 tagged_above=-999 required=5\r
+       tests=[BAYES_00=-1.9, T_MIME_NO_TEXT=0.01] autolearn=ham\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 53foZeBlAG4m for <notmuch@notmuchmail.org>;\r
+       Thu, 29 Apr 2010 02:00:23 -0700 (PDT)\r
+Received: from mail-ww0-f53.google.com (mail-ww0-f53.google.com\r
+ [74.125.82.53])       by olra.theworths.org (Postfix) with ESMTP id 715B2431FC1       for\r
+ <notmuch@notmuchmail.org>; Thu, 29 Apr 2010 02:00:23 -0700 (PDT)\r
+Received: by wwe15 with SMTP id 15so553682wwe.26\r
+       for <notmuch@notmuchmail.org>; Thu, 29 Apr 2010 02:00:22 -0700 (PDT)\r
+Received: by 10.216.157.202 with SMTP id o52mr8731124wek.212.1272531618222;\r
+       Thu, 29 Apr 2010 02:00:18 -0700 (PDT)\r
+Received: from ut.hh.sledj.net (gmp-ea-fw-1b.sun.com [192.18.8.1])\r
+       by mx.google.com with ESMTPS id x1sm5583894wbx.7.2010.04.29.02.00.16\r
+       (version=TLSv1/SSLv3 cipher=RC4-MD5);\r
+       Thu, 29 Apr 2010 02:00:17 -0700 (PDT)\r
+Received: by ut.hh.sledj.net (Postfix, from userid 1000)\r
+       id E629159411F; Thu, 29 Apr 2010 10:00:37 +0100 (BST)\r
+To: Sebastian Spaeth <Sebastian@SSpaeth.de>,\r
+       Notmuch developer list <notmuch@notmuchmail.org>\r
+Subject: Re: [PATCH] emacs: Reuse rather than reinvent message header\r
+ filtering\r
+In-Reply-To: <1272530875-7141-1-git-send-email-Sebastian@SSpaeth.de>\r
+References: <1272530875-7141-1-git-send-email-Sebastian@SSpaeth.de>\r
+User-Agent: Notmuch/0.3.1-24-gac275ad (http://notmuchmail.org) Emacs/23.1.1\r
+       (x86_64-pc-linux-gnu)\r
+From: David Edmondson <dme@dme.org>\r
+Date: Thu, 29 Apr 2010 10:00:34 +0100\r
+Message-ID: <87eihyy6od.fsf@ut.hh.sledj.net>\r
+MIME-Version: 1.0\r
+Content-Type: multipart/signed; boundary="=-=-=";\r
+       micalg=pgp-sha1; protocol="application/pgp-signature"\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: Thu, 29 Apr 2010 09:00:24 -0000\r
+\r
+--=-=-=\r
+Content-Transfer-Encoding: quoted-printable\r
+\r
+On Thu, 29 Apr 2010 10:47:55 +0200, Sebastian Spaeth <Sebastian@SSpaeth.de>=\r
+ wrote:\r
+> In notmuch-mua-reply we were filtering out the Subject and To headers\r
+> manually in a loop, but message mode offers a nice function for\r
+> exactly that. Simplify the code by using that. Also, as\r
+> notmuch-mua-mail already sorts and hides headers that we want sorted\r
+> and hidden, we can safely remove those 2 functions from here as well.\r
+>=20\r
+> Signed-off-by: Sebastian Spaeth <Sebastian@SSpaeth.de>\r
+\r
+This looks fine to me (I even tested!).\r
+\r
+We can also remove "(require 'cl)" now that the use of `loop' is gone.\r
+\r
+dme.\r
+=2D-=20\r
+David Edmondson, http://dme.org\r
+\r
+--=-=-=\r
+Content-Type: application/pgp-signature\r
+\r
+-----BEGIN PGP SIGNATURE-----\r
+Version: GnuPG v1.4.10 (GNU/Linux)\r
+\r
+iEYEARECAAYFAkvZSrIACgkQaezQq/BJZRa0HwCfS0UmF7s+QK43nEW4XSUTfch9\r
+fLMAn2+uy7JAZ+1Gpp7RS+3DPdGakKQQ\r
+=1yBG\r
+-----END PGP SIGNATURE-----\r
+--=-=-=--\r