about summary refs log tree commit homepage
path: root/lib/PublicInbox/SearchView.pm
diff options
context:
space:
mode:
authorEric Wong <e@80x24.org>2021-10-13 07:00:36 +0000
committerEric Wong <e@80x24.org>2021-10-13 19:51:20 +0000
commit96b0a14be7e62742ad06f0a37c3cba61fe6c51e7 (patch)
tree2b1fcabfb1b23804c0d05424303f589d1d59a0f2 /lib/PublicInbox/SearchView.pm
parent6bb8478c8dacf83492de3afe15a615d646f0904f (diff)
downloadpublic-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.pm7
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};