On Wed, Jul 19, 2017 at 09:08:16 -0400, John Ferlan wrote: > > > On 07/11/2017 08:56 AM, Peter Krempa wrote: > > Make sure that JSON strings can contain characters which need to be > > escaped (double quotes, backslashes, tabs, etc.) and that JSON objects > > formatted into strings can be nested into strings. > > --- > > tests/virjsontest.c | 66 +++++++++++++++++++++++++++++++++++++++++++++++++++++ > > 1 file changed, 66 insertions(+) > > > > Seems there should be two separate patches here, one that ensures > escaped strings work for DO_TEST_PARSE and the other passing/using a Okay I'll split it and resend. > nested string obj. Is "EscapeObj" even a right name, seems like > "NestedObj" would be more appropriate. Is there some sort of escaping in > the strings that I'm missing? The naming issue is somewhere between those two, since you can nest JSON naturally, but nesting a json string in a json value is what we are tesing here. > > Also, w/r/t escaping do we need handle double commas ",," that are > supposed to turn into "," ? Commas? Aren't you confusing this with escaping for qemu? JSON escaping rules don't have anything special for commas since the escaping needs to be done only if you want to store a JSON string into another JSON string. Commas are not interpreted inside of strings.
Description: PGP signature