util: escape C1 control characters and switch to hex format (!29826) · Merge requests · Rodrigo Test / Test Group-nodejs / node · GitLab

Do not update/delete: Banner broadcast message test data

Do not update/delete: Notification broadcast message test data

Rodrigo Muino Tomonari requested to merge github/fork/BridgeAR/escape-c1-controls into master Oct 03, 2019

C1 control characters will from now on also be escaped to prevent altering the terminal behavior.

This also switches '\u00XX' to '\xXX' as suggested by @dd-pardal. The reason for me to switch is that it uses less characters.

Fixes: https://github.com/nodejs/node/issues/29450

Checklist

make -j4 test (UNIX), or vcbuild test (Windows) passes
tests and/or benchmarks are included
documentation is changed or added
commit message follows commit guidelines