From: Jesse Rosenthal <jrosenthal@jhu.edu>
To: David Edmondson <dme@dme.org>
Subject: Re: [PATCH] notmuch-addresses: Match on the full name as well as
	components.
In-Reply-To: <8739c17ues.fsf@jhu.edu>
References: <1324475357-21700-1-git-send-email-dme@dme.org>
	<8739c17ues.fsf@jhu.edu>
User-Agent: Notmuch/0.10.2 (http://notmuchmail.org) Emacs/23.3.1
	(i686-pc-linux-gnu)
Date: Fri, 30 Dec 2011 14:38:10 -0500
Message-ID: <87wr9d6eal.fsf@jhu.edu>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Cc: notmuch@notmuchmail.org
Precedence: list

Oh, and I moved the code to

http://commonmeasure.org/~jkr/notmuch_addresses.git

Best,
Jesse

On Fri, 30 Dec 2011 14:04:43 -0500, Jesse Rosenthal <jrosenthal@jhu.edu> wrote:
> Pushed.
> 
> Thanks,
> Jesse
> 
> On Wed, 21 Dec 2011 13:49:17 +0000, David Edmondson <dme@dme.org> wrote:
> > ---
> >  notmuch_addresses.py |    7 +++++--
> >  1 files changed, 5 insertions(+), 2 deletions(-)
> > 
> > diff --git a/notmuch_addresses.py b/notmuch_addresses.py
> > index bf45151..74a743c 100755
> > --- a/notmuch_addresses.py
> > +++ b/notmuch_addresses.py
> > @@ -164,11 +164,14 @@ class NotmuchAddressMatcher(object):
> >                      addrs.append(v)
> >                      parsed_addrs = email.utils.getaddresses(addrs)
> >              for addr in parsed_addrs:
> > -                mail = addr[1].lower()
> > -                split_names = addr[0].split(" ")
> > +                full_name = addr[0]
> > +                split_names = full_name.split(" ")
> > +                mail = addr[1]
> >                  if (len([name for name in split_names 
> >                           if self.match_function(name)]) > 0
> >                      or 
> > +                    self.match_function(full_name)
> > +                    or 
> >                      self.match_function(mail)):
> >                      
> >                      emails.add_email_and_name(mail, addr[0])
> > -- 
> > 1.7.7.3
> >