]> Sergey Matveev's repositories - public-inbox.git/commitdiff
nntpdeflate: stop relying on SUPER for ->do_read
authorEric Wong <e@80x24.org>
Sat, 13 Jul 2019 20:27:57 +0000 (20:27 +0000)
committerEric Wong <e@80x24.org>
Sat, 13 Jul 2019 21:49:40 +0000 (21:49 +0000)
We won't need further layering after enabling compression.  So
be explicit about which sub we're calling when we hit ->do_read
from NNTP and eliminate the need for the comment.

lib/PublicInbox/NNTPdeflate.pm

index 10e2337cda97ba1fa83fcecc28fdfd97592c92b3..f2de0f381c470f6b8c616aa43aff424fae2c65cc 100644 (file)
@@ -62,13 +62,13 @@ sub enable {
 # overrides PublicInbox::NNTP::compressed
 sub compressed { 1 }
 
-# SUPER is PublicInbox::DS::do_read, so $_[1] may be a reference or not
+# $_[1] may be a reference or not
 sub do_read ($$$$) {
        my ($self, $rbuf, $len, $off) = @_;
 
        my $zin = $self->{zin} or return; # closed
        my $deflated = \($zin->[1]);
-       my $r = $self->SUPER::do_read($deflated, $len) or return;
+       my $r = PublicInbox::DS::do_read($self, $deflated, $len) or return;
 
        # assert(length($$rbuf) == $off) as far as NNTP.pm is concerned
        # -ConsumeInput is true, so $deflated is automatically emptied