www: use PerlIO::scalar (zfh) for buffering

[public-inbox.git] / lib / PublicInbox / WwwAtomStream.pm

diff --git a/lib/PublicInbox/WwwAtomStream.pm b/lib/PublicInbox/WwwAtomStream.pm
index 09b6facbb337da0119ec063524a5dbd06373a323..33da32443c0f941cb01ce6de07a625b03d176097 100644 (file)
--- a/lib/PublicInbox/WwwAtomStream.pm
+++ b/lib/PublicInbox/WwwAtomStream.pm
@@ -1,4 +1,4 @@
-# Copyright (C) 2016-2021 all contributors <meta@public-inbox.org>
+# Copyright (C) all contributors <meta@public-inbox.org>
 # License: AGPL-3.0+ <https://www.gnu.org/licenses/agpl-3.0.txt>
 #
 # Atom body stream for HTTP responses
@@ -43,10 +43,10 @@ sub async_eml { # for async_blob_cb
 }
 
 sub response {
-       my ($class, $ctx, $code, $cb) = @_;
+       my ($class, $ctx, $cb) = @_;
        my $res_hdr = [ 'Content-Type' => 'application/atom+xml' ];
        $class->new($ctx, $cb);
-       $ctx->psgi_response($code, $res_hdr);
+       $ctx->psgi_response(200, $res_hdr);
 }
 
 # called once for each message by PSGI server
@@ -146,20 +146,19 @@ sub feed_entry {
        my $name = ascii_html(join(', ', PublicInbox::Address::names($from)));
        $email = ascii_html($email // $ctx->{ibx}->{-primary_address});
 
-       my $s = delete($ctx->{emit_header}) ? atom_header($ctx, $title) : '';
-       $s .= "<entry><author><name>$name</name><email>$email</email>" .
+       print { $ctx->zfh }
+               (delete($ctx->{emit_header}) ? atom_header($ctx, $title) : ''),
+               "<entry><author><name>$name</name><email>$email</email>" .
                "</author>$title$updated" .
-               qq(<link\nhref="$href"/>).
+               qq(<link\nhref="$href"/>) .
                "<id>$uuid</id>$irt" .
                qq{<content\ntype="xhtml">} .
                qq{<div\nxmlns="http://www.w3.org/1999/xhtml">} .
                qq(<pre\nstyle="white-space:pre-wrap">);
-       $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 {