Re: [Qemu-devel] [PATCH v5 00/15] Add clone visitor

[Markus Armbruster]

Markus Armbruster <address@hidden> writes:

> Eric Blake <address@hidden> writes:
>
>> [First half of v4 00/28 Add qapi-to-JSON and clone visitors:
>> https://lists.gnu.org/archive/html/qemu-devel/2016-05/msg03220.html]
>>
>> No hard prerequisites; applies to master
>>
>> Soft prerequisites (for valgrind to be happy with all touched tests):
>> My fix for memleak in range.h (still waiting for other reviewers to
>> chime in on what semantics we want documented in range.h):
>> https://lists.gnu.org/archive/html/qemu-devel/2016-05/msg02666.html
>> My fix for memleak in json-streamer.c, in Markus' qapi-next branch:
>> https://lists.gnu.org/archive/html/qemu-devel/2016-05/msg03186.html
>>
>> Also available as a tag at:
>> git fetch git://repo.or.cz/qemu/ericb.git qapi-jsonv5
>> [yes, it's a misnomer now that I split the series in two, but
>> matches the v4 tag name]
>>
>> Differences since v4:
>> - drop old patch 1, all includes of a qjson.h now have a directory
>> prefix making it obvious which include is intended
>> - split patch 11/28
>> - rebase remaining patches
>> - various comment improvements, based on Markus' review
>> - fix clone visitor crash
>> - tweak clone visitor handling of NULL where "" was intended
>
> Series
> Reviewed-by: Markus Armbruster <address@hidden>
>
> Since I'd prefer to commit your "[PATCH v2 0/3] Fix leak in handling of
> integer lists as strings" to qapi-next first, I committed this series to
> qapi-not-next for now.

Graduated to qapi-next.  A bit more patience...