about summary refs log tree commit homepage
path: root/HACKING
diff options
context:
space:
mode:
authorEric Wong <e@80x24.org>2021-02-06 12:18:38 +0000
committerEric Wong <e@80x24.org>2021-02-07 03:34:32 +0000
commita2bae6437eeb8147736812eca95e85e5642c97cc (patch)
tree9aba5ac1c9f07f090a8788755f6e30c6ad371270 /HACKING
parent7f43cdaf0e46c1d99bf5834f414dc8f36407e8cc (diff)
downloadpublic-inbox-a2bae6437eeb8147736812eca95e85e5642c97cc.tar.gz
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/
Diffstat (limited to 'HACKING')
0 files changed, 0 insertions, 0 deletions