Re: [Qemu-devel] [PATCH 28/56] json: Fix \uXXXX for surrogate pairs

2018-08-13 Thread Markus Armbruster
Paolo Bonzini writes: > On 08/08/2018 14:03, Markus Armbruster wrote: >> +if (cp >= 0xD800 && cp <= 0xDBFF && !leading_surrogate >> +&& ptr[1] == '\\' && ptr[2] == 'u') { >> +ptr += 2; >> +leading_surrogate = cp; >> +

Re: [Qemu-devel] [PATCH 28/56] json: Fix \uXXXX for surrogate pairs

2018-08-13 Thread Markus Armbruster
Eric Blake writes: > On 08/08/2018 07:03 AM, Markus Armbruster wrote: >> The JSON parser treats each half of a surrogate pair as unpaired >> surrogate. Fix it to recognize surrogate pairs. >> >> Signed-off-by: Markus Armbruster >> --- >> qobject/json-parser.c | 16 +++- >> tests/

Re: [Qemu-devel] [PATCH 28/56] json: Fix \uXXXX for surrogate pairs

2018-08-12 Thread Paolo Bonzini
On 08/08/2018 14:03, Markus Armbruster wrote: > +if (cp >= 0xD800 && cp <= 0xDBFF && !leading_surrogate > +&& ptr[1] == '\\' && ptr[2] == 'u') { > +ptr += 2; > +leading_surrogate = cp; > +goto hex; > +

Re: [Qemu-devel] [PATCH 28/56] json: Fix \uXXXX for surrogate pairs

2018-08-10 Thread Eric Blake
On 08/08/2018 07:03 AM, Markus Armbruster wrote: The JSON parser treats each half of a surrogate pair as unpaired surrogate. Fix it to recognize surrogate pairs. Signed-off-by: Markus Armbruster --- qobject/json-parser.c | 16 +++- tests/check-qjson.c | 3 +-- 2 files change

[Qemu-devel] [PATCH 28/56] json: Fix \uXXXX for surrogate pairs

2018-08-08 Thread Markus Armbruster
The JSON parser treats each half of a surrogate pair as unpaired surrogate. Fix it to recognize surrogate pairs. Signed-off-by: Markus Armbruster --- qobject/json-parser.c | 16 +++- tests/check-qjson.c | 3 +-- 2 files changed, 16 insertions(+), 3 deletions(-) diff --git a/qobj