Escaped strings in JSON

[shreyasbharath]

Hello again,

Say if we were to parse JSON that looked like this -

{
    "message": "\"Hello there\""
}

When we read it out like so -

JsonObject root = parser->parse( json );
const char *message = root["message"];
// Shouldn't we be getting the unescaped form here?

I noticed that we still get the escaped form. Perhaps this is a bug in Jsmn?

[bblanchon]

I do agree there is something wrong here: you expected message to be "Hello there" but it's actually \"Hello there\"
The backslashes should not be present in the output.

But there is a good reason for that
As it's a library designed for low-end embedded system, I originally decided that the library would not be allowed to move bytes in the string.
I thought that the performance penality wasn't worth it and that nobody would ever see it.

Turns out I was wrong twice.
First you proved me that somebody did see the bug.
Second, I now think that the cost of moving is acceptable, it’s just O(1) after all.

That's going to be in 3.4.
Thanks again 😉

[shreyasbharath]

Awesome :)

[bblanchon]

OK, it's fixed in 3.4.

The code size increased by 118 bytes on 8-bit AVR.
I think that's acceptable.

Sorry it took so long for such a small fix.