]> Sergey Matveev's repositories - public-inbox.git/blobdiff - lib/PublicInbox/SearchMsg.pm
Merge commit 'mem'
[public-inbox.git] / lib / PublicInbox / SearchMsg.pm
index 292efceed7c012bc36cee85fd79a5eb5795c6399..ceb6edadded21431bf2ca3793b254ee0b1fcd7b7 100644 (file)
@@ -3,6 +3,8 @@
 # based on notmuch, but with no concept of folders, files or flags
 #
 # Wraps a document inside our Xapian search index.
+# There may be many of these objects loaded in memory at once
+# for large threads in our WWW UI.
 package PublicInbox::SearchMsg;
 use strict;
 use warnings;
@@ -82,18 +84,21 @@ sub load_expand {
        $self;
 }
 
+sub psgi_cull ($) {
+       my ($self) = @_;
+       from_name($self); # fill in {from_name} so we can delete {from}
+
+       # drop NNTP-only fields which aren't relevant to PSGI results:
+       # saves ~80K on a 200 item search result:
+       delete @$self{qw(from ts to cc bytes lines)};
+       $self;
+}
+
 # Only called by PSGI interface, not NNTP
 sub load_doc {
        my ($class, $doc) = @_;
        my $self = bless {}, $class;
-       my $smsg = load_expand($self, $doc);
-
-       from_name($smsg); # fill in {from_name} so we can delete {from}
-
-       # drop NNTP-only fields which aren't relevant to PSGI results:
-       # saves ~80K on a 200 item search result:
-       delete @$smsg{qw(from ts to cc bytes lines)};
-       $smsg;
+       psgi_cull(load_expand($self, $doc));
 }
 
 # :bytes and :lines metadata in RFC 3977