From: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Date: Wed, 20 Apr 2011 22:50:51 GMT
Subject: Re: [RFC PATCH] git-send-email: Validate recipient_cmd (to-cmd, cc-cmd) addresses
Message-ID: <BANLkTintK1KgPWX7ycMQtVBnKi+tVj7k2Q@mail.gmail.com>
In-Reply-To: <1303339511.24766.41.camel@Joe-Laptop>

On Thu, Apr 21, 2011 at 00:45, Joe Perches <joe@perches.com> wrote:
> On Thu, 2011-04-21 at 00:29 +0200, Ævar Arnfjörð Bjarmason wrote:
>> On Wed, Apr 20, 2011 at 23:50, Joe Perches <joe@perches.com> wrote:
>> > +       my ($recipient_name, $recipient_addr) = ($recipient =~ /^\s*(.*?)\s*(<[^>]+>)/);
>> In Perl you can write (<.*?>) instead of (<[^>]+>)
>
> Hey Ævar.  That matches <>.  Not a good email address.

True, but you can use <.+?> instead.

I meant that you don't need to work around the lack of non-greedy
regex features in Perl.