about summary refs log tree commit homepage
path: root/lib/PublicInbox/LEI.pm
diff options
context:
space:
mode:
authorEric Wong <e@80x24.org>2023-10-04 03:49:22 +0000
committerEric Wong <e@80x24.org>2023-10-04 17:46:38 +0000
commitb3538311b66310cb160569e5bdbcb00e91f4b9aa (patch)
treeae3f028a545494ea725ef666dc8d7b02bad6d038 /lib/PublicInbox/LEI.pm
parenta51576acbab6445687c407eba4615b01933d8bc3 (diff)
downloadpublic-inbox-b3538311b66310cb160569e5bdbcb00e91f4b9aa.tar.gz
No need to define our own empty `noop' sub when PublicInbox::Config
already has one and is loaded anyways.
Diffstat (limited to 'lib/PublicInbox/LEI.pm')
-rw-r--r--lib/PublicInbox/LEI.pm14
1 files changed, 3 insertions, 11 deletions
diff --git a/lib/PublicInbox/LEI.pm b/lib/PublicInbox/LEI.pm
index fba4edf3..c9ad46e2 100644
--- a/lib/PublicInbox/LEI.pm
+++ b/lib/PublicInbox/LEI.pm
@@ -1217,8 +1217,6 @@ sub event_step_init {
         };
 }
 
-sub noop {}
-
 sub oldset { $oldset }
 
 sub dump_and_clear_log {
@@ -1364,15 +1362,9 @@ sub lazy_start {
                         $lis->close; # DS::close
                 };
         };
-        my $sig = {
-                CHLD => \&PublicInbox::DS::enqueue_reap,
-                QUIT => $quit,
-                INT => $quit,
-                TERM => $quit,
-                HUP => \&noop,
-                USR1 => \&noop,
-                USR2 => \&noop,
-        };
+        my $sig = { CHLD => \&PublicInbox::DS::enqueue_reap };
+        $sig->{$_} = $quit for qw(QUIT INT TERM);
+        $sig->{$_} = \&PublicInbox::Config::noop for qw(HUP USR1 USR2);
         # for EVFILT_SIGNAL and signalfd behavioral difference:
         my @kq_ign = eval { require PublicInbox::DSKQXS } ? keys(%$sig) : ();