This project is archived and is in readonly mode.
Simplify regex for filtering non-ASCII chars in JSON escape
#2850
A regex in JSON escape seems to be too much complex. I even can't see whether it works as expected.
Attached a patch to simply filter out the non-ASCII chars.
Please ignore this patch if I misunderstood the spec, but it seems that the tests passes anyway.
-
json_escape_regex_fix.diff
1.13 KB
-
Yehuda Katz (wycats)- State changed from new to invalid
It looks like the original regex was there for a reason. Do you have any reason to believe it's just arbitrary?