Re: [RFC] reordering and cleanup of thread authors

diff --git a/eb/f8417730452a2448b411593b60dc6321b23ce6 b/eb/f8417730452a2448b411593b60dc6321b23ce6
new file mode 100644 (file)
index 0000000..961403e
--- /dev/null
+++ b/eb/f8417730452a2448b411593b60dc6321b23ce6
@@ -0,0 +1,86 @@
+Return-Path: <BATV+bbc4756b788e67be553a+2433+infradead.org+hohndel@bombadil.srs.infradead.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 CE12F4196F2\r
+       for <notmuch@notmuchmail.org>; Wed, 21 Apr 2010 20:57:20 -0700 (PDT)\r
+X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
+X-Spam-Flag: NO\r
+X-Spam-Score: -2.8\r
+X-Spam-Level: \r
+X-Spam-Status: No, score=-2.8 tagged_above=-999 required=5\r
+       tests=[BAYES_05=-0.5, RCVD_IN_DNSWL_MED=-2.3] 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 0WOumiqwf-3X for <notmuch@notmuchmail.org>;\r
+       Wed, 21 Apr 2010 20:57:19 -0700 (PDT)\r
+Received: from bombadil.infradead.org (bombadil.infradead.org [18.85.46.34])\r
+       by olra.theworths.org (Postfix) with ESMTP id D27CF431FC1\r
+       for <notmuch@notmuchmail.org>; Wed, 21 Apr 2010 20:57:19 -0700 (PDT)\r
+Received: from localhost ([::1] helo=x200.gr8dns.org)\r
+       by bombadil.infradead.org with esmtp (Exim 4.69 #1 (Red Hat Linux))\r
+       id 1O4nXT-0006wL-Bw; Thu, 22 Apr 2010 03:57:19 +0000\r
+Received: by x200.gr8dns.org (Postfix, from userid 500)\r
+       id C76F9C0212; Wed, 21 Apr 2010 20:57:18 -0700 (PDT)\r
+From: Dirk Hohndel <hohndel@infradead.org>\r
+To: Michal Sojka <sojkam1@fel.cvut.cz>, notmuch <notmuch@notmuchmail.org>\r
+Subject: Re: [RFC] reordering and cleanup of thread authors\r
+In-Reply-To: <m3wrwg6n9y.fsf@x200.gr8dns.org>\r
+References: <m31veru7vn.fsf@x200.gr8dns.org> <87zl1d5fc0.fsf@steelpick.2x.cz>\r
+       <m3zl1cfsb4.fsf@x200.gr8dns.org>\r
+       <87aatcysw8.fsf@wsheee.localdomain>\r
+       <m3wrwg6n9y.fsf@x200.gr8dns.org>\r
+Date: Wed, 21 Apr 2010 20:57:18 -0700\r
+Message-ID: <m3aaswyw9t.fsf@x200.gr8dns.org>\r
+User-Agent: notmuch 0.2-52-ga28d2fe (Emacs 23.1.1/i386-redhat-linux-gnu)\r
+MIME-Version: 1.0\r
+Content-Type: text/plain; charset=us-ascii\r
+X-SRS-Rewrite: SMTP reverse-path rewritten from <hohndel@infradead.org> by\r
+       bombadil.infradead.org See http://www.infradead.org/rpr.html\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, 22 Apr 2010 03:57:21 -0000\r
+\r
+On Fri, 09 Apr 2010 19:42:49 -0700, Dirk Hohndel <hohndel@infradead.org> wrote:\r
+> On Sat, 10 Apr 2010 03:53:59 +0200, Michal Sojka <sojkam1@fel.cvut.cz> wrote:\r
+> > I think that using | as a separator would help here. Let's say that\r
+> > initially we have "Matched Author, Non Matched, Matched Again" we can\r
+> > tranform this to  "Matched Author, Matched Again| Non Matched". This way,\r
+> > the length of the string remains the same. If there is no | after\r
+> > transformation, we know that all authors matched because there is always\r
+> > at least one mathed author in the search results.\r
+> \r
+> That's a great idea. I'll update the patch to do that.\r
+\r
+Since Carl just prompted me, I wrote an updated patch (will post in a\r
+separate message replying to this one). \r
+\r
+I made the change suggested by Michal, fixed a bug or two and removed\r
+the part of the patch that was trying to cleanup author names in "Last,\r
+First" format - on IRC it was pointed out to me that I was overlooking\r
+another use of the ',' in email addresses: accounts that are shared by\r
+multiple people. And it makes no sense to reorder email addresses of the\r
+form "Wife, Husband and child" <familyaccount@add.res>\r
+\r
+I haven't given up on this, though. Since Exchange and Outlook have this\r
+nasty habit of creating these "Last, First" <first.last@company.com> or\r
+"Last, First MI" <first.mi.last@company.com> from headers, I really want\r
+to add the option to clean those up. So I'll submit a separate patch\r
+that checks if we have exactly one of these two pattern shown here - and\r
+that then reorders things\r
+\r
+/D\r
+\r
+-- \r
+Dirk Hohndel\r
+Intel Open Source Technology Center\r