From: Felipe Contreras Date: Sun, 19 May 2013 15:45:58 GMT Subject: Re: [PATCH v5 01/15] Add new git-related helper to contrib Message-ID: In-Reply-To: On Sun, May 19, 2013 at 10:17 AM, Ramkumar Ramachandra wrote: > Ramkumar Ramachandra wrote: >> There's a non-optional space before the "" in your regex, which >> is what I was pointing out. > > Er, scratch that. It's the space after the "Whatevered-by:" It doesn't really matter. We can operate under the assumption that there will always be a name and remove the extra code in fmt_person(), IIRC it was because of the muttrc simplification which I don't think makes sense any more. It's still possible that the name will be missing, but I don't personally care, it would probably work for 99.99% of the cases. -- Felipe Contreras