diff options
| author |  Ævar Arnfjörð Bjarmason <avarab@gmail.com> | 2010-09-30 13:43:02 +0000 |
|---|---|---|
| committer |  Junio C Hamano <gitster@pobox.com> | 2010-09-30 12:20:33 -0700 |
| commit | ff483897317b5379e189d6bd782270dd08c09966 (patch) | |
| tree | 500c5a88d51d420e0e6572fab8a1d8e7c0bac10d | |
| parent | send-email: use \E***\Q instead of \*\*\* (diff) | |
| download | tgif-ff483897317b5379e189d6bd782270dd08c09966.tar.xz | |
send-email: sanitize_address use $foo, not "$foo"
There's no reason to explicitly stringify a variable in Perl unless
it's an overloaded object and you want to call overload::StrVal,
otherwise it's just creating a new scalar redundantly.
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.comReviewed-by: Avery Pennarun <apenwarr@gmail.com>
Reviewed-by: Jeff King <peff@peff.net>
>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
| -rwxr-xr-x | git-send-email.perl | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/git-send-email.perl b/git-send-email.perl index 1218bbe729..1bf090a881 100755 --- a/git-send-email.perl +++ b/git-send-email.perl @@ -864,7 +864,7 @@ sub sanitize_address { my ($recipient_name, $recipient_addr) = ($recipient =~ /^(.*?)\s*(<.*)/); if (not $recipient_name) { - return "$recipient"; + return $recipient; } # if recipient_name is already quoted, do nothing |