From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.3.2 (2011-06-06) on dcvr.yhbt.net X-Spam-Level: X-Spam-ASN: X-Spam-Status: No, score=-2.9 required=3.0 tests=ALL_TRUSTED,AWL,BAYES_00, RP_MATCHES_RCVD shortcircuit=no autolearn=unavailable version=3.3.2 X-Original-To: meta@public-inbox.org Received: from localhost (dcvr.yhbt.net [127.0.0.1]) by dcvr.yhbt.net (Postfix) with ESMTP id D2F5963382B for ; Sat, 5 Mar 2016 06:07:44 +0000 (UTC) From: Eric Wong To: meta@public-inbox.org Subject: [PATCH 4/4] daemon: simplify parent death handling Date: Sat, 5 Mar 2016 06:07:40 +0000 Message-Id: <20160305060740.13472-5-e@80x24.org> In-Reply-To: <20160305060740.13472-1-e@80x24.org> References: <20160305060740.13472-1-e@80x24.org> List-Id: No need to create a new sub which kill ourselves $$ when we can invoke worker_quit directly. --- lib/PublicInbox/Daemon.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/PublicInbox/Daemon.pm b/lib/PublicInbox/Daemon.pm index 30411e1..8a0af8d 100644 --- a/lib/PublicInbox/Daemon.pm +++ b/lib/PublicInbox/Daemon.pm @@ -391,7 +391,7 @@ sub daemon_loop ($$) { $refresh->(); # preload by default $parent_pipe = master_loop(); # returns if in child process my $fd = fileno($parent_pipe); - Danga::Socket->AddOtherFds($fd => sub { kill('TERM', $$) } ); + Danga::Socket->AddOtherFds($fd => *worker_quit); } else { reopen_logs(); $set_user->() if $set_user; -- EW