[PATCH 1/2] wwwlisting: use first successfully loaded JSON module

[Eric Wong]

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")
---
 lib/PublicInbox/WwwListing.pm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

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;
        }
 }
 
--
unsubscribe: one-click, see List-Unsubscribe header
archive: https://public-inbox.org/meta/