Skip to content

src,test: fix JSON escaping in node-report

Rodrigo Muino Tomonari requested to merge github/fork/lundibundi/fix-json-escaping into master

Previously only simple escape sequences were handled (i.e. \n, \t, r etc.). This commit adds escaping of other control symbols in the range of 0x00 to 0x20.

Also, this replaces multiple find+replace calls with a single pass replacer.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • commit message follows commit guidelines

Merge request reports