about summary refs log tree commit homepage
diff options
context:
space:
mode:
-rw-r--r--lib/PublicInbox/View.pm15
-rw-r--r--lib/PublicInbox/WwwAtomStream.pm5
2 files changed, 6 insertions, 14 deletions
diff --git a/lib/PublicInbox/View.pm b/lib/PublicInbox/View.pm
index 37b484ae..0b67d92f 100644
--- a/lib/PublicInbox/View.pm
+++ b/lib/PublicInbox/View.pm
@@ -39,8 +39,8 @@ sub msg_page_i {
                 $ctx->{mhref} = ($ctx->{nr} || $ctx->{smsg}) ?
                                 "../${\mid_href($smsg->{mid})}/" : '';
                 if (_msg_page_prepare_obuf($eml, $ctx)) {
-                        multipart_text_as_html($eml, $ctx);
-                        ${$ctx->{obuf}} .= '</pre><hr>';
+                        $eml->each_part(\&add_text_body, $ctx, 1);
+                        $ctx->zmore('</pre><hr>');
                 }
                 html_footer($ctx, $ctx->{first_hdr}) if !$ctx->{smsg};
                 delete($ctx->{obuf}) // \'';
@@ -57,8 +57,8 @@ sub no_over_html ($) {
         $ctx->{mhref} = '';
         PublicInbox::WwwStream::init($ctx);
         if (_msg_page_prepare_obuf($eml, $ctx)) { # sets {-title_html}
-                multipart_text_as_html($eml, $ctx);
-                ${$ctx->{obuf}} .= '</pre><hr>';
+                $eml->each_part(\&add_text_body, $ctx, 1);
+                $ctx->zmore('</pre><hr>');
         }
         html_footer($ctx, $eml);
         $ctx->html_done;
@@ -506,13 +506,6 @@ sub thread_html_i { # PublicInbox::WwwStream::getline callback
         }
 }
 
-sub multipart_text_as_html {
-        # ($mime, $ctx) = @_; # each_part may do "$_[0] = undef"
-
-        # scan through all parts, looking for displayable text
-        $_[0]->each_part(\&add_text_body, $_[1], 1);
-}
-
 sub submsg_hdr ($$) {
         my ($ctx, $eml) = @_;
         my $obfs_ibx = $ctx->{-obfs_ibx};
diff --git a/lib/PublicInbox/WwwAtomStream.pm b/lib/PublicInbox/WwwAtomStream.pm
index 906b292a..09c79a8a 100644
--- a/lib/PublicInbox/WwwAtomStream.pm
+++ b/lib/PublicInbox/WwwAtomStream.pm
@@ -157,9 +157,8 @@ sub feed_entry {
         $ctx->{obuf} = \$s;
         $ctx->{mhref} = $href;
         $ctx->{changed_href} = "${href}#related";
-        PublicInbox::View::multipart_text_as_html($eml, $ctx);
-        delete $ctx->{obuf};
-        $s .= '</pre></div></content></entry>';
+        $eml->each_part(\&PublicInbox::View::add_text_body, $ctx, 1);
+        '</pre></div></content></entry>';
 }
 
 sub feed_updated {