about summary refs log tree commit homepage
path: root/lib/PublicInbox/SearchIdx.pm
diff options
context:
space:
mode:
authorEric Wong <e@yhbt.net>2020-04-18 03:38:53 +0000
committerEric Wong <e@yhbt.net>2020-04-19 08:51:29 +0000
commit0c586dc64b3b6642a894e125d09df446667a4079 (patch)
tree38d6b349b64e6395e1910aaae2ef450b06d96a13 /lib/PublicInbox/SearchIdx.pm
parent92de3139920992bfad32ef927153f27addfdc72c (diff)
downloadpublic-inbox-0c586dc64b3b6642a894e125d09df446667a4079.tar.gz
It's unnecessary overhead for anything which does Email::MIME
parsing.  It was never done for v2 indexing, even though v1->v2
conversions did NOT remove those From_ lines.  There was never a
need to remote From_ lines the v1 SearchIdx paths, either.

Hitting a /$INBOX_URL/$MSGID/T/ endpoint with an 18 message
thread reveals a ~0.5% speed improvement.  This will become
more apparent when we have a faster MIME parser.
Diffstat (limited to 'lib/PublicInbox/SearchIdx.pm')
-rw-r--r--lib/PublicInbox/SearchIdx.pm2
1 files changed, 0 insertions, 2 deletions
diff --git a/lib/PublicInbox/SearchIdx.pm b/lib/PublicInbox/SearchIdx.pm
index d1290dc2..579b85e3 100644
--- a/lib/PublicInbox/SearchIdx.pm
+++ b/lib/PublicInbox/SearchIdx.pm
@@ -553,8 +553,6 @@ sub do_cat_mail {
         my ($git, $blob, $sizeref) = @_;
         my $str = $git->cat_file($blob, $sizeref) or
                 die "BUG: $blob not found in $git->{git_dir}";
-        # fixup bugs from import:
-        $$str =~ s/\A[\r\n]*From [^\r\n]*\r?\n//s;
         PublicInbox::MIME->new($str);
 }