diff options
author | Eric Wong <e@80x24.org> | 2021-09-03 08:54:22 +0000 |
---|---|---|
committer | Eric Wong <e@80x24.org> | 2021-09-03 08:55:05 +0000 |
commit | a73b85cc93e635e5f8a0520ab0f21177fb8de478 (patch) | |
tree | 9394d9858dd62941584cb7482f8586aebec375b9 /lib/PublicInbox/LeiLcat.pm | |
parent | 0ea8c77b4339398131889afd2405b5e10abd8f57 (diff) | |
download | public-inbox-a73b85cc93e635e5f8a0520ab0f21177fb8de478.tar.gz |
lei: ->child_error less error-prone
I was calling "child_error(1, ...)" in a few places where I meant to be calling "child_error(1 << 8, ...)" and inadvertantly triggering SIGHUP in script/lei. Since giving a zero exit code to child_error makes no sense, just allow falsy values to default to 1 << 8.
Diffstat (limited to 'lib/PublicInbox/LeiLcat.pm')
-rw-r--r-- | lib/PublicInbox/LeiLcat.pm | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/PublicInbox/LeiLcat.pm b/lib/PublicInbox/LeiLcat.pm index 9d95e899..1e54c3bf 100644 --- a/lib/PublicInbox/LeiLcat.pm +++ b/lib/PublicInbox/LeiLcat.pm @@ -18,7 +18,7 @@ sub lcat_folder ($$$) { my $err = $lms->arg2folder($lei, $folders); $lei->qerr(@{$err->{qerr}}) if $err && $err->{qerr}; if ($err && $err->{fail}) { - $lei->child_error(1 << 8, "# unknown folder: $folder"); + $lei->child_error(0, "# unknown folder: $folder"); } else { for my $f (@$folders) { my $fid = $lms->fid_for($f); |