From: Eric Wong <e@yhbt.net>
Date: Sat, 21 Mar 2020 01:10:33 +0000 (+0000)
Subject: wwwlisting: use first successfully loaded JSON module
X-Git-Tag: v1.4.0~76
X-Git-Url: http://www.git.stargrave.org/?a=commitdiff_plain;h=9ba72fb4d832390eac423e25e6734a9b21086322;p=public-inbox.git

wwwlisting: use first successfully loaded JSON module

And not the last...

I only noticed this since JSON::PP::Boolean was spewing
redefinition warnings via overload.pm

Fixes: 8fb8fc52420ef669 ("wwwlisting: avoid lazy loading JSON module")
---

diff --git a/lib/PublicInbox/WwwListing.pm b/lib/PublicInbox/WwwListing.pm
index 33cb0ace..42a0c0d8 100644
--- a/lib/PublicInbox/WwwListing.pm
+++ b/lib/PublicInbox/WwwListing.pm
@@ -20,7 +20,7 @@ if (eval { require IO::Compress::Gzip }) {
 	for my $mod (qw(JSON::MaybeXS JSON JSON::PP)) {
 		eval "require $mod" or next;
 		# ->ascii encodes non-ASCII to "\uXXXX"
-		$json = $mod->new->ascii(1);
+		$json = $mod->new->ascii(1) and last;
 	}
 }