diff options
author | Eric Wong <e@80x24.org> | 2021-10-13 07:00:36 +0000 |
---|---|---|
committer | Eric Wong <e@80x24.org> | 2021-10-13 19:51:20 +0000 |
commit | 96b0a14be7e62742ad06f0a37c3cba61fe6c51e7 (patch) | |
tree | 2b1fcabfb1b23804c0d05424303f589d1d59a0f2 /lib/PublicInbox/SearchView.pm | |
parent | 6bb8478c8dacf83492de3afe15a615d646f0904f (diff) | |
download | public-inbox-96b0a14be7e62742ad06f0a37c3cba61fe6c51e7.tar.gz |
treewide: use warn() or carp() instead of env->{psgi.errors}
Large chunks of our codebase and 3rd-party dependencies do not use ->{psgi.errors}, so trying to standardize on it was a fruitless endeavor. Since warn() and carp() are standard mechanism within Perl, just use that instead and simplify a bunch of existing code.
Diffstat (limited to 'lib/PublicInbox/SearchView.pm')
-rw-r--r-- | lib/PublicInbox/SearchView.pm | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/lib/PublicInbox/SearchView.pm b/lib/PublicInbox/SearchView.pm index e74ddb90..a42867c5 100644 --- a/lib/PublicInbox/SearchView.pm +++ b/lib/PublicInbox/SearchView.pm @@ -102,11 +102,8 @@ sub mset_summary { foreach my $m ($mset->items) { my $num = shift @nums; - my $smsg = delete($num2msg{$num}) or do { - eval { - $m = "$m $num expired\n"; - $ctx->{env}->{'psgi.errors'}->print($m); - }; + my $smsg = delete($num2msg{$num}) // do { + warn "$m $num expired\n"; next; }; my $mid = $smsg->{mid}; |