From: Tait Date: Tue, 11 Nov 2008 01:49:19 GMT Subject: Re: [PATCH] git send-email: edit recipient addresses with the --compose flag Message-ID: <20081111014919.GI7408@ece.pdx.edu> In-Reply-To: > > > + if ($c_file =~ /^To:\s*+(.+)\s*\nCc:/ism) { > > > > Greedy operators are only supported with perl 5.10 or more... I think it's a > > bad idea to use them... > > The problem here was that a space should follow the field, but it may > not. The user may unwarily backup over it. "\s*" would match this > case. > > But if there is a space, it is included in the "(.+)". Not in any version of Perl to which I have access. Why doesn't /^To:\s*(.+)\s*\nCc:/ism work?