send-email: minor code refactoring
authorRemi Lespinet <remi.lespinet@ensimag.grenoble-inp.fr>
Tue, 30 Jun 2015 12:16:47 +0000 (14:16 +0200)
committerJunio C Hamano <gitster@pobox.com>
Tue, 30 Jun 2015 18:34:35 +0000 (11:34 -0700)
Group expressions in a single if statement. This avoid checking
multiple time if the variable $sender is defined.

Signed-off-by: Remi Lespinet <remi.lespinet@ensimag.grenoble-inp.fr>
Signed-off-by: Matthieu Moy <Matthieu.Moy@imag.fr>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
git-send-email.perl
index 228400b3e937b28f6bc57211ffe6409f42721dbb..09ecad830f3075b3b0dfe3f9ce7db396c9adeb2c 100755 (executable)
@@ -774,9 +774,9 @@ sub file_declares_8bit_cte {
        }
 }
 
-($sender) = expand_aliases($sender) if defined $sender;
-
-if (!defined $sender) {
+if (defined $sender) {
+       ($sender) = expand_aliases($sender);
+} else {
        $sender = $repoauthor || $repocommitter || '';
 }