diff options
author | Eric Wong <e@80x24.org> | 2023-11-02 09:35:28 +0000 |
---|---|---|
committer | Eric Wong <e@80x24.org> | 2023-11-02 23:46:54 +0000 |
commit | db47ea523760f7ceef3d38c0008e082e9e7d1c02 (patch) | |
tree | c16385b1824df5bc506fbe4c983e8fe3a31d0a48 /lib/PublicInbox/DS.pm | |
parent | 52cd1dd47c83f18681bab081db5041f8bfa5dfbe (diff) | |
download | public-inbox-db47ea523760f7ceef3d38c0008e082e9e7d1c02.tar.gz |
treewide: use ->close method rather than CORE::close
It's easier-to-read and should open the door for us to get rid of `tie' for ProcessIO without performance penalties for more frequently-used perlop calls and ability to do `stat' directly on the object instead of the awkward `tied' thing.
Diffstat (limited to 'lib/PublicInbox/DS.pm')
-rw-r--r-- | lib/PublicInbox/DS.pm | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/PublicInbox/DS.pm b/lib/PublicInbox/DS.pm index 8331da95..33f80087 100644 --- a/lib/PublicInbox/DS.pm +++ b/lib/PublicInbox/DS.pm @@ -341,8 +341,8 @@ sub greet { my $ev = EPOLLIN; my $wbuf; if ($sock->can('accept_SSL') && !$sock->accept_SSL) { - return CORE::close($sock) if $! != EAGAIN; - $ev = PublicInbox::TLS::epollbit() or return CORE::close($sock); + return $sock->close if $! != EAGAIN; + $ev = PublicInbox::TLS::epollbit() or return $sock->close; $wbuf = [ \&accept_tls_step, $self->can('do_greet')]; } new($self, $sock, $ev | EPOLLONESHOT); |