rurban / Cpanel-JSON-XS

Improved fork of JSON-XS

Home Page:http://search.cpan.org/dist/Cpanel-JSON-XS/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

allow_singlequote should also allow \'

warpspin opened this issue · comments

In my opinion, if allow_singlequote option is enabled, it should also be possible to escape it using a backslash sequence. Right now it will error with a "illegal backslash escape sequence in string" error.

With allow_singlequote disabled, \' should stay a "illegal backslash escape sequence in string", though, as it currently is, to stay JSON standard compliant.

test2.pl.txt

care for a patch?

Ok, sent you a pull request with a patch.