user/dev discussion of public-inbox itself
 help / color / mirror / code / Atom feed
* [PATCH] manifest: favor Cpanel::JSON::XS
@ 2020-10-03 23:12 Eric Wong
  0 siblings, 0 replies; only message in thread
From: Eric Wong @ 2020-10-03 23:12 UTC (permalink / raw)
  To: meta

JSON::MaybeXS already favors Cpanel::JSON::XS (and has for many
years, now).  Allow users to skip installing JSON::MaybeXS if
they want an XS-based JSON implementation.
---
 There is more JSON on the way...

 lib/PublicInbox/ManifestJsGz.pm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/PublicInbox/ManifestJsGz.pm b/lib/PublicInbox/ManifestJsGz.pm
index f98d9d01..16d2a87c 100644
--- a/lib/PublicInbox/ManifestJsGz.pm
+++ b/lib/PublicInbox/ManifestJsGz.pm
@@ -16,7 +16,7 @@ use HTTP::Date qw(time2str);
 *try_cat = \&PublicInbox::Inbox::try_cat;
 
 our $json;
-for my $mod (qw(JSON::MaybeXS JSON JSON::PP)) {
+for my $mod (qw(Cpanel::JSON::XS JSON::MaybeXS JSON JSON::PP)) {
 	eval "require $mod" or next;
 	# ->ascii encodes non-ASCII to "\uXXXX"
 	$json = $mod->new->ascii(1) and last;

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2020-10-03 23:12 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-10-03 23:12 [PATCH] manifest: favor Cpanel::JSON::XS Eric Wong

Code repositories for project(s) associated with this inbox:

	https://80x24.org/public-inbox.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).