Jzon icon indicating copy to clipboard operation
Jzon copied to clipboard

Fix for escaped characters

Open warvair opened this issue 10 years ago • 0 comments

I discovered that control characters were not escaped properly when written to file. It seems to me that as long as strings are in memory (C strings or std::string) the control codes should not be escaped, they should only be escaped when written to a stream and unescaped when parsed from a stream. So that's where I put the calls.

This may or may not be the best fix for this issue, but it seems to work.

Also, the test I added is a bit wonky and you might want to change it.

My editor whitespace settings seem to be incompatible with yours or github's, so I reverted and recommitted with less changes to whitespace (by making targeted edits) but there's still seems to be alignment issues. Anyway, that's why there are 3 commits.

warvair avatar May 30 '15 20:05 warvair