Felipe Contreras <felipe.contreras@xxxxxxxxx> writes: > On Mon, Nov 19, 2012 at 11:58 PM, Krzysztof Mazur <krzysiek@xxxxxxxxxxxx> wrote: > >> --- a/git-send-email.perl >> +++ b/git-send-email.perl >> @@ -924,6 +924,10 @@ sub quote_subject { >> # use the simplest quoting being able to handle the recipient >> sub sanitize_address { >> my ($recipient) = @_; >> + >> + # remove garbage after email address >> + $recipient =~ s/(.*>).*$/$1/; >> + > > Looks fine, but I would do s/(.*?>)(.*)$/$1/, so that 'test > <foo@xxxxxxx> <#comment>' gets the second comment removed. Yeah, but do you need to capture the second group? IOW, like "s/(.*?>).*$/$1/" perhaps? -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html