The existing code mixes parsing of email header with regular
expression and actual code. Extract the parsing code into a new
subroutine 'parse_header_line()'. This improves the code readability
and make parse_header_line reusable in other place.
Signed-off-by: Nathan Payre <nathan.payre@xxxxxxxxxxxxxxxxx>
Signed-off-by: Matthieu Moy <matthieu.moy@xxxxxxxxxxxxx>
Signed-off-by: Timothee Albertin <timothee.albertin@xxxxxxxxxxxxxxxxx>
Signed-off-by: Daniel Bensoussan <daniel.bensoussan--bohm@xxxxxxxxxxxxxxxxx>
Thanks-to: Ævar Arnfjörð Bjarmason <avarab@xxxxxxxxx>
---
Without the "print" used for testing.
git-send-email.perl | 90 +++++++++++++++++++++++++++++++++++++----------------
1 file changed, 63 insertions(+), 27 deletions(-)
diff --git a/git-send-email.perl b/git-send-email.perl
index 2208dcc21..a10574a56 100755
--- a/git-send-email.perl
+++ b/git-send-email.perl
@@ -715,41 +715,63 @@ EOT3
if (!defined $compose_encoding) {
$compose_encoding = "UTF-8";
}
- while(<$c>) {
- next if m/^GIT:/;
- if ($in_body) {
- $summary_empty = 0 unless (/^\n$/);
- } elsif (/^\n$/) {
- $in_body = 1;
- if ($need_8bit_cte) {
+
+ my %parsed_email;
+ $parsed_email{'body'} = '';
+ while (my $line = <$c>) {
+ next if $line =~ m/^GIT:/;
+ parse_header_line($line, \%parsed_email);
+ if ($line =~ /^\n$/i) {
+ while (my $body_line = <$c>) {
+ if ($body_line !~ m/^GIT:/) {
+ $parsed_email{'body'} = $parsed_email{'body'} . $body_line;
+ }
+ }
+ }
+ }
+
+ if ($parsed_email{'from'}) {
+ $sender = $parsed_email{'from'};
+ }
+ if ($parsed_email{'in-reply-to'}) {
+ $initial_reply_to = $parsed_email{'in-reply-to'};
+ }
+ if ($parsed_email{'subject'}) {
+ $initial_subject = $parsed_email{'subject'};
+ print $c2 "Subject: " .
+ quote_subject($parsed_email{'subject'}, $compose_encoding) .
+ "\n";
+ }
+ if ($parsed_email{'mime-version'}) {
+ $need_8bit_cte = 0;
+ print $c2 "MIME-Version: $parsed_email{'mime-version'}\n",
+ "Content-Type: $parsed_email{'content-type'};\n",
+ "Content-Transfer-Encoding: $parsed_email{'content-transfer-encoding'}\n";
+ }
+ if ($need_8bit_cte) {
+ if ($parsed_email{'content-type'}) {
+ print $c2 "MIME-Version: 1.0\n",
+ "Content-Type: $parsed_email{'content-type'};",
+ "Content-Transfer-Encoding: 8bit\n";
+ } else {
print $c2 "MIME-Version: 1.0\n",
"Content-Type: text/plain; ",
- "charset=$compose_encoding\n",
+ "charset=$compose_encoding\n",
"Content-Transfer-Encoding: 8bit\n";
}
- } elsif (/^MIME-Version:/i) {
- $need_8bit_cte = 0;
- } elsif (/^Subject:\s*(.+)\s*$/i) {
- $initial_subject = $1;
- my $subject = $initial_subject;
- $_ = "Subject: " .
- quote_subject($subject, $compose_encoding) .
- "\n";
- } elsif (/^In-Reply-To:\s*(.+)\s*$/i) {
- $initial_reply_to = $1;
- next;
- } elsif (/^From:\s*(.+)\s*$/i) {
- $sender = $1;
- next;
- } elsif (/^(?:To|Cc|Bcc):/i) {
- print __("To/Cc/Bcc fields are not interpreted yet, they have been ignored\n");
- next;
- }
- print $c2 $_;
}
+ if ($parsed_email{'body'}) {
+ $summary_empty = 0;
+ print $c2 "\n$parsed_email{'body'}\n";
+ }
+
close $c;
close $c2;
+ open $c2, "<", $compose_filename . ".final"
+ or die sprintf(__("Failed to open %s.final: %s"), $compose_filename, $!);
+ close $c2;
+
if ($summary_empty) {
print __("Summary email is empty, skipping it\n");
$compose = -1;
@@ -792,6 +814,20 @@ sub ask {
return;
}
+sub parse_header_line {
+ my $lines = shift;
+ my $parsed_line = shift;
+
+ foreach (split(/\n/, $lines)) {
+ if (/^(To|Cc|Bcc):\s*(.+)$/i) {
+ $parsed_line->{lc $1} = [ parse_address_line($2) ];
+ } elsif (/^(From|Subject|Date|In-Reply-To|Message-ID|MIME-Version|Content-Type|Content-Transfer-Encoding|References):\s*(.+)\s*$/i) {
+ $parsed_line->{lc $1} = $2;
+ }
+ }
+}
+
+
my %broken_encoding;
sub file_declares_8bit_cte {
--
2.15.1
