qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH 09/56] check-qjson: Cover escaped characters mor


From: Markus Armbruster
Subject: Re: [Qemu-devel] [PATCH 09/56] check-qjson: Cover escaped characters more thoroughly, part 2
Date: Fri, 10 Aug 2018 16:16:20 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/26.1 (gnu/linux)

Eric Blake <address@hidden> writes:

> On 08/08/2018 07:02 AM, Markus Armbruster wrote:
>> Cover surrogates, invalid escapes, and noncharacters.  This
>> demonstrates that valid surrogate pairs are misinterpreted, and
>> invalid surrogates and noncharacters aren't rejected.
>>
>> Signed-off-by: Markus Armbruster <address@hidden>
>> ---
>>   tests/check-qjson.c | 53 ++++++++++++++++++++++++++++++++++++++-------
>>   1 file changed, 45 insertions(+), 8 deletions(-)
>>
>
>> +        { "\\u12x", NULL },
>> +        { "\\u123x", NULL },
>> +        { "\\u12345", "\341\210\2645" },
>> +        { "\\u12345", "\341\210\2645" },
>
> Why is this one duplicated?

Editing accident, fixing.

> Otherwise,
> Reviewed-by: Eric Blake <address@hidden>

Thanks!



reply via email to

[Prev in Thread] Current Thread [Next in Thread]