]> git.ipfire.org Git - thirdparty/public-inbox.git/commitdiff
ds: reap_pids: remove redundant signal blocking
authorEric Wong <e@80x24.org>
Wed, 15 Mar 2023 21:47:56 +0000 (21:47 +0000)
committerEric Wong <e@80x24.org>
Thu, 16 Mar 2023 19:07:17 +0000 (19:07 +0000)
Blocking signals when reaping was done when the lei pager was
spawned by the daemon in b90e8d6e02.  Shortly afterwards in
7b79c918a5, the client script took over spawning of the pager
and made b90e8d6e02 redundant.

cf. b90e8d6e02 (ds: block signals when reaping, 2021-01-10)
    7b79c918a5 (lei: run pager in client script, 2021-01-10)

lib/PublicInbox/DS.pm

index a08e01f540bb34e07689030a0880035ce2f193d2..b6eaf2d7563f7a011c20cc27d39ee6faec65b28e 100644 (file)
@@ -214,7 +214,6 @@ sub await_cb ($;@) {
 # that to remain the case.
 sub reap_pids {
        $reap_armed = undef;
-       my $oldset = block_signals();
        while (1) {
                my $pid = waitpid(-1, WNOHANG) // last;
                last if $pid <= 0;
@@ -224,7 +223,6 @@ sub reap_pids {
                        warn "W: reaped unknown PID=$pid: \$?=$?\n";
                }
        }
-       sig_setmask($oldset);
 }
 
 # reentrant SIGCHLD handler (since reap_pids is not reentrant)