about summary refs log tree commit homepage
path: root/lib/PublicInbox/DS.pm
diff options
context:
space:
mode:
authorEric Wong <e@80x24.org>2023-11-25 20:54:35 +0000
committerEric Wong <e@80x24.org>2023-11-26 01:05:26 +0000
commit8517e1fc5ec052719920c11c6babce2a395c48a9 (patch)
treea02263561d5e4cb46f592082bc32d99d13f09e78 /lib/PublicInbox/DS.pm
parent85f161be11a0c5561aa872fcadff3d08b8163ff6 (diff)
downloadpublic-inbox-8517e1fc5ec052719920c11c6babce2a395c48a9.tar.gz
We already stash the associated FD for reporting at startup and
don't need to call `fileno' again.  Found via manual code
inspection while considering the effort to make async {forward}
from PublicInbox::HTTP more like the generic long_response API
and {long_cb} field used by IMAP/NNTP/POP3.
Diffstat (limited to 'lib/PublicInbox/DS.pm')
-rw-r--r--lib/PublicInbox/DS.pm1
1 files changed, 0 insertions, 1 deletions
diff --git a/lib/PublicInbox/DS.pm b/lib/PublicInbox/DS.pm
index 4c8b502f..8bc8cfb7 100644
--- a/lib/PublicInbox/DS.pm
+++ b/lib/PublicInbox/DS.pm
@@ -629,7 +629,6 @@ sub long_step {
                 delete $self->{long_cb};
                 $self->long_response_done;
                 my $elapsed = now() - $t0;
-                my $fd = fileno($self->{sock});
                 $self->out(" deferred[$fd] done - %0.6f", $elapsed);
                 my $wbuf = $self->{wbuf}; # do NOT autovivify
                 requeue($self) unless $wbuf && @$wbuf;