about summary refs log tree commit homepage
path: root/lib/PublicInbox/InboxWritable.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/InboxWritable.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/InboxWritable.pm')
-rw-r--r--lib/PublicInbox/InboxWritable.pm2
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/PublicInbox/InboxWritable.pm b/lib/PublicInbox/InboxWritable.pm
index 31eb3f15..b1d5caf5 100644
--- a/lib/PublicInbox/InboxWritable.pm
+++ b/lib/PublicInbox/InboxWritable.pm
@@ -292,7 +292,7 @@ sub warn_ignore {
 
 # this expects to be RHS in this assignment: "local $SIG{__WARN__} = ..."
 sub warn_ignore_cb {
-        my $cb = $SIG{__WARN__} // sub { print STDERR @_ };
+        my $cb = $SIG{__WARN__} // \&CORE::warn;
         sub {
                 return if warn_ignore(@_);
                 $cb->(@_);