send-email: use the three-arg form of open in recipients_cmd
commita47eab03f613fa55b9e690d5354e95bc165dceee
authorRamkumar Ramachandra <artagnon@gmail.com>
Mon, 1 Apr 2013 01:40:42 +0000 (31 18:40 -0700)
committerJunio C Hamano <gitster@pobox.com>
Mon, 1 Apr 2013 04:30:27 +0000 (31 21:30 -0700)
tree3a73ed82b67fc7e08c327514d4653af4e8daba3d
parent9b39703920b2d64985abcc1348b169d8fa658c24
send-email: use the three-arg form of open in recipients_cmd

Perlcritic does not want to see the trailing pipe in the two-args
form of open(), i.e.

open my $fh, "$cmd \Q$file\E |";

If $cmd were a single-token command name, it would make a lot more
sense to use four-or-more-args form "open FILEHANDLE,MODE,CMD,ARGS"
to avoid shell from expanding metacharacters in $file, but we do
expect multi-word string in $to_cmd and $cc_cmd to be expanded by
the shell, so we cannot rewrite it to

open my $fh, "-|", $cmd, $file;

for extra safety.  At least, by using this in the three-arg form:

open my $fh, "-|", "$cmd \Q$file\E";

we can silence Perlcritic, even though we do not gain much safety by
doing so.

Signed-off-by: Ramkumar Ramachandra <artagnon@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
git-send-email.perl