about summary refs log tree commit homepage
path: root/lib/PublicInbox/Admin.pm
diff options
context:
space:
mode:
authorEric Wong <e@80x24.org>2020-12-26 01:44:36 +0000
committerEric Wong <e@80x24.org>2020-12-26 19:27:15 +0000
commit0b018bebe7d8ee807ab07b570cf33669da4875b0 (patch)
tree9a0eb73298a8226b722d81db1d88a2fdb7d7adcf /lib/PublicInbox/Admin.pm
parentb5e960f50289434025f5904c8c1311e4c8a02b82 (diff)
downloadpublic-inbox-0b018bebe7d8ee807ab07b570cf33669da4875b0.tar.gz
As with CORE::die and $SIG{__DIE__}, it turns out CORE::warn is
safe to use inside $SIG{__WARN__} handlers without triggering
infinite recursion.  So fall back to reusing CORE::warn instead
of creating a new sub.
Diffstat (limited to 'lib/PublicInbox/Admin.pm')
-rw-r--r--lib/PublicInbox/Admin.pm2
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/PublicInbox/Admin.pm b/lib/PublicInbox/Admin.pm
index b468108e..d414e4e2 100644
--- a/lib/PublicInbox/Admin.pm
+++ b/lib/PublicInbox/Admin.pm
@@ -241,7 +241,7 @@ sub index_inbox {
         }
         local %SIG = %SIG;
         setup_signals(\&index_terminate, $ibx);
-        my $warn_cb = $SIG{__WARN__} // sub { print STDERR @_ };
+        my $warn_cb = $SIG{__WARN__} // \&CORE::warn;
         my $idx = { current_info => $ibx->{inboxdir} };
         my $warn_ignore = PublicInbox::InboxWritable->can('warn_ignore');
         local $SIG{__WARN__} = sub {