reduce scope of mbox From_ line removal

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.

1 files changed, 2 insertions, 6 deletions

diff --git a/lib/PublicInbox/Inbox.pm b/lib/PublicInbox/Inbox.pm
index 4bd82989..186eb420 100644
--- a/lib/PublicInbox/Inbox.pm
+++ b/lib/PublicInbox/Inbox.pm
@@ -311,9 +311,7 @@ sub nntp_usable {
 # for v1 users w/o SQLite only
 sub msg_by_path ($$;$) {
         my ($self, $path, $ref) = @_;
-        my $str = git($self)->cat_file('HEAD:'.$path, $ref);
-        $$str =~ s/\A[\r\n]*From [^\r\n]*\r?\n//s if $str;
-        $str;
+        git($self)->cat_file('HEAD:'.$path, $ref);
 }
 
 sub msg_by_smsg ($$;$) {
@@ -324,9 +322,7 @@ sub msg_by_smsg ($$;$) {
         return unless defined $smsg;
         defined(my $blob = $smsg->{blob}) or return;
 
-        my $str = git($self)->cat_file($blob, $ref);
-        $$str =~ s/\A[\r\n]*From [^\r\n]*\r?\n//s if $str;
-        $str;
+        git($self)->cat_file($blob, $ref);
 }
 
 sub smsg_mime {