From 8517e1fc5ec052719920c11c6babce2a395c48a9 Mon Sep 17 00:00:00 2001 From: Eric Wong Date: Sat, 25 Nov 2023 20:54:35 +0000 Subject: ds: long_step: eliminate redundant fileno call 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. --- lib/PublicInbox/DS.pm | 1 - 1 file changed, 1 deletion(-) (limited to 'lib/PublicInbox') 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; -- cgit v1.2.3-24-ge0c7