]> Sergey Matveev's repositories - public-inbox.git/commitdiff
daemon: use POSIX and WNOHANG more idiomatically
authorEric Wong <e@80x24.org>
Sun, 7 Jul 2019 03:49:19 +0000 (03:49 +0000)
committerEric Wong <e@80x24.org>
Mon, 8 Jul 2019 07:29:26 +0000 (07:29 +0000)
No point in uglifying our code since we need the POSIX
module in many places, anyways.

lib/PublicInbox/Daemon.pm

index 2046a7f5abffd382d8a79f2e7aa2c5ee1d923432..6cb3a0caa912390c0b3a75c3d93588a80b0fc008 100644 (file)
@@ -8,6 +8,7 @@ use warnings;
 use Getopt::Long qw/:config gnu_getopt no_ignore_case auto_abbrev/;
 use IO::Handle;
 use IO::Socket;
+use POSIX qw(WNOHANG);
 use Socket qw(IPPROTO_TCP SOL_SOCKET);
 sub SO_ACCEPTFILTER () { 0x1000 }
 use Cwd qw/abs_path/;
@@ -15,7 +16,6 @@ STDOUT->autoflush(1);
 STDERR->autoflush(1);
 use PublicInbox::DS qw(now);
 require PublicInbox::EvCleanup;
-require POSIX;
 require PublicInbox::Listener;
 require PublicInbox::ParentPipe;
 my @CMD;
@@ -437,7 +437,7 @@ sub upgrade_aborted ($) {
 
 sub reap_children () {
        while (1) {
-               my $p = waitpid(-1, &POSIX::WNOHANG) or return;
+               my $p = waitpid(-1, WNOHANG) or return;
                if (defined $reexec_pid && $p == $reexec_pid) {
                        upgrade_aborted($p);
                } elsif (defined(my $id = delete $pids{$p})) {