about summary refs log tree commit homepage
path: root/lib/PublicInbox/SearchView.pm
diff options
authorEric Wong <e@yhbt.net>2020-06-01 10:06:52 +0000
committerEric Wong <e@yhbt.net>2020-06-03 04:04:21 +0000
commit7e782e8ff3ec64500913069a2efb5815fe711a36 (patch)
tree52ac6cf6ac5a8bbfae97ffd6cae2f160e4ee22d4 /lib/PublicInbox/SearchView.pm
parentd32333a12ca46861fab82640c3697fdbca879bd2 (diff)
To further simplify callers and avoid embarrasing memory
explosions[1], we can finally eliminate this method in
favor of smsg_eml.

[1] commit 7d02b9e64455831d3bda20cd2e64e0c15dc07df5
    ("view: stop storing all MIME objects on large threads")
    fixed a huge memory blowup.
Diffstat (limited to 'lib/PublicInbox/SearchView.pm')
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/PublicInbox/SearchView.pm b/lib/PublicInbox/SearchView.pm
index 249cf539..d53a533e 100644
--- a/lib/PublicInbox/SearchView.pm
+++ b/lib/PublicInbox/SearchView.pm
@@ -300,8 +300,8 @@ sub mset_thread_i {
         my ($nr, $ctx) = @_;
         my $msgs = $ctx->{msgs} or return;
         while (my $smsg = pop @$msgs) {
-                $ctx->{-inbox}->smsg_mime($smsg) or next;
-                return PublicInbox::View::index_entry($smsg, $ctx,
+                my $eml = $ctx->{-inbox}->smsg_eml($smsg) or next;
+                return PublicInbox::View::eml_entry($ctx, $smsg, $eml,
                                                         scalar @$msgs);
         my ($skel) = delete @$ctx{qw(skel msgs)};