ds: favor `delete' over assigning fields to `undef'

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

diff --git a/lib/PublicInbox/NNTP.pm b/lib/PublicInbox/NNTP.pm
index fbdf1364ccb80423ea3f6811668148e737f26a65..6a582ea41397412253afb5503f79a23610c78c16 100644 (file)
--- a/lib/PublicInbox/NNTP.pm
+++ b/lib/PublicInbox/NNTP.pm
@@ -624,7 +624,7 @@ sub long_response ($$) {
        $self->{long_res} = sub {
                my $more = eval { $cb->() };
                if ($@ || !$self->{sock}) { # something bad happened...
-                       $self->{long_res} = undef;
+                       delete $self->{long_res};
 
                        if ($@) {
                                err($self,
@@ -646,7 +646,7 @@ sub long_response ($$) {
                        push @$nextq, $self;
                        $nextt ||= PublicInbox::EvCleanup::asap(*next_tick);
                } else { # all done!
-                       $self->{long_res} = undef;
+                       delete $self->{long_res};
                        check_read($self);
                        res($self, '.');
                        out($self, " deferred[$fd] done - %0.6f", now() - $t0);