about summary refs log tree commit homepage
path: root/lib/PublicInbox/LeiViewText.pm
diff options
context:
space:
mode:
authorEric Wong <e@80x24.org>2021-10-06 11:19:36 +0000
committerEric Wong <e@80x24.org>2021-10-06 11:24:35 +0000
commita80fd0c8e19464f95dec5a5c2d128a1614f51ee5 (patch)
tree51195ef50ba851a56db0969a47d558e437a94d1a /lib/PublicInbox/LeiViewText.pm
parente785573c556572a675407796604e9c0a9965ea9f (diff)
downloadpublic-inbox-a80fd0c8e19464f95dec5a5c2d128a1614f51ee5.tar.gz
The regexp in split_quotes relies on the presence of a
final "\n", so add it wherever we need to instead of
making it the responsibility of every caller.

This probably doesn't matter in practice since every
email seems to have a "\n" as the final byte (due to
the way SMTP works), but maybe there's some odd ones
that'll get imported via lei.
Diffstat (limited to 'lib/PublicInbox/LeiViewText.pm')
-rw-r--r--lib/PublicInbox/LeiViewText.pm1
1 files changed, 0 insertions, 1 deletions
diff --git a/lib/PublicInbox/LeiViewText.pm b/lib/PublicInbox/LeiViewText.pm
index 1f002ccd..c469d1ea 100644
--- a/lib/PublicInbox/LeiViewText.pm
+++ b/lib/PublicInbox/LeiViewText.pm
@@ -245,7 +245,6 @@ sub add_text_buf { # callback for Eml->each_part
         hdr_buf($self, $part) if $part->{is_submsg};
         $s =~ s/\r\n/\n/sg;
         _xs($s);
-        $s .= "\n" unless substr($s, -1, 1) eq "\n";
         my $diff = ($s =~ /^--- [^\n]+\n\+{3} [^\n]+\n@@ /ms);
         my @sections = PublicInbox::MsgIter::split_quotes($s);
         undef $s; # free memory