Re: [Qemu-devel] [PATCH 12/56] check-qjson: Simplify utf8_string()

[Eric Blake]

On 08/08/2018 07:02 AM, Markus Armbruster wrote:
> The previous commit made utf8_string()'s test_cases[].utf8_in
> superfluous: we can use .json_in instead.  Except for the case testing
> U+0000.  \x00 doesn't work in C strings, so it tests \\u0000 instead.
> But testing \\uXXXX is escaped_string()'s job.  It's covered there.
> Test U+0001 here, and drop .utf8_in.
>
> Signed-off-by: Markus Armbruster <address@hidden>
> ---
>   tests/check-qjson.c | 53 ++++++++-------------------------------------
>   1 file changed, 9 insertions(+), 44 deletions(-)

Reviewed-by: Eric Blake <address@hidden>

--
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3266
Virtualization:  qemu.org | libvirt.org