]> Sergey Matveev's repositories - public-inbox.git/commit
treewide: replace confess with croak
authorEric Wong <e@80x24.org>
Sat, 6 Feb 2021 12:18:38 +0000 (12:18 +0000)
committerEric Wong <e@80x24.org>
Sun, 7 Feb 2021 03:34:32 +0000 (03:34 +0000)
commita2bae6437eeb8147736812eca95e85e5642c97cc
tree9aba5ac1c9f07f090a8788755f6e30c6ad371270
parent7f43cdaf0e46c1d99bf5834f414dc8f36407e8cc
treewide: replace confess with croak

The PublicInbox::Eml (and previously Email::MIME) use of confess
was the primary (or only) culprit behind the lei2mail segfaults
fixed by commit 0795b0906cc81f40.
("ds: guard against stack-not-refcounted quirk of Perl 5").

We never care about a backtrace when dealing with Eml objects
anyways, so it was just a worthless waste of CPU cycles.

We can also drop confess in a few other places.  Since we only
use Perl and Inline::C, users will never be without source
and can replace s/croak/Carp::confess/ on a per-callsite basis
to help report problems.

It's also possible to use PERL5OPT=-MCarp=verbose in the
environment though still potentially risky.

Link: https://public-inbox.org/meta/20210201082833.3293-1-e@80x24.org/
lib/PublicInbox/DS.pm
lib/PublicInbox/Eml.pm
lib/PublicInbox/IPC.pm
lib/PublicInbox/OverIdx.pm