Re: [PATCH] send-email: improve recipients_cmd()

From: Junio C Hamano <gitster@pobox.com>
Date: Apr 29, 2013, 08:15 UTC
Message-ID: <7vhaipn45f.fsf@alter.siamese.dyndns.org>
In-Reply-To: <CAMP44s3QkiQhYr7pKHgKFoXH5m9_ZHLHmwC4c9WC2OGjv7LHdg@mail.gmail.com>

Felipe Contreras <felipe.contreras@gmail.com> writes:

Show 7 quoted lines

>> so if we want to get rid of \Q\E, it would be:
>>
>>         open $fh, '-|', qw(sh -c), "$cmd " . '"$@"', '-', @args
>>
>
> I don't know if that would be better, or converting @args to a list of
> quoted strings, essentially keeping the current behavior.

I tend to agree, so let's drop it for now.