From mboxrd@z Thu Jan  1 00:00:00 1970
From: marcus-sup@bar-coded.net (Marcus Williams)
Date: Fri, 25 Apr 2008 22:15:38 +0100
Subject: [sup-talk] [PATCH] Add list-id as a new property of messages
In-Reply-To: <1209156188-sup-868@elly>
References: <1208985725-sup-6028@elly> <1209026545-sup-9368@tomsk>
	<1209156188-sup-868@elly>
Message-ID: <1209157604-sup-2682@tomsk>

On 25.4.2008, Israel Herraiz wrote:
> Yes, I know. The "problem" is that List-Id use to have two fields: one
> that is a name, and another one that is a address-like. I wanted to
> use that last field and no the name for the automatic label. That's
> why I have used PersonManager to process the List-Id header.

:) I'm not saying you shouldnt do it - just that personally I dont
think you should do it in the place you do it. If you make list-id the
raw header in your patch then you can do the person manager lookup in
your before-add-message labeling hook (assuming thats where you're
labeling stuff). That way someone that wants to do something different
with the list-id header can do because you've given them the raw
header contents. 

Having said all that, you can access the raw headers directly from the
message object in the hook anyway so maybe this is a non-issue. I'll
shut up :)

Marcus