diff options
author | Eric Wong <e@80x24.org> | 2023-11-25 20:54:35 +0000 |
---|---|---|
committer | Eric Wong <e@80x24.org> | 2023-11-26 01:05:26 +0000 |
commit | 8517e1fc5ec052719920c11c6babce2a395c48a9 (patch) | |
tree | a02263561d5e4cb46f592082bc32d99d13f09e78 /lib/PublicInbox/DS.pm | |
parent | 85f161be11a0c5561aa872fcadff3d08b8163ff6 (diff) | |
download | public-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.pm | 1 |
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; |