Re: [RFC PATCH 1/2] send-email: fix garbage removal after address

2017-08-25 Thread Matthieu Moy
Junio C Hamano writes: > Matthieu Moy writes: > >> +sub strip_garbage_one_address { >> +my ($addr) = @_; >> +chomp $addr; >> +if ($addr =~ /^(("[^"]*"|[^"<]*)? *<[^>]*>).*/) { >> +# "Foo Bar" [possibly garbage here] >> +# Foo Bar [possibly garbage here] >> +

Re: [RFC PATCH 1/2] send-email: fix garbage removal after address

2017-08-24 Thread Junio C Hamano
Matthieu Moy writes: > This is a followup over 9d33439 (send-email: only allow one address > per body tag, 2017-02-20). The first iteration did allow writting > > Cc: # garbage > > but did so by matching the regex ([^>]*>?), i.e. stop after the first > instance of '>'. However, it did not prop

Re: [RFC PATCH 1/2] send-email: fix garbage removal after address

2017-08-23 Thread Jacob Keller
On Wed, Aug 23, 2017 at 3:21 AM, Matthieu Moy wrote: > This is a followup over 9d33439 (send-email: only allow one address > per body tag, 2017-02-20). The first iteration did allow writting > > Cc: # garbage > > but did so by matching the regex ([^>]*>?), i.e. stop after the first > instance o

[RFC PATCH 1/2] send-email: fix garbage removal after address

2017-08-23 Thread Matthieu Moy
This is a followup over 9d33439 (send-email: only allow one address per body tag, 2017-02-20). The first iteration did allow writting Cc: # garbage but did so by matching the regex ([^>]*>?), i.e. stop after the first instance of '>'. However, it did not properly deal with Cc: f...@example.