[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v5 00/15] Add clone visitor
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH v5 00/15] Add clone visitor |
Date: |
Mon, 13 Jun 2016 16:19:27 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
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.
- [Qemu-devel] [PATCH v5 15/15] replay: Use new QAPI cloning, (continued)
- [Qemu-devel] [PATCH v5 15/15] replay: Use new QAPI cloning, Eric Blake, 2016/06/09
- [Qemu-devel] [PATCH v5 06/15] string-input-visitor: Favor new visit_free() function, Eric Blake, 2016/06/09
- [Qemu-devel] [PATCH v5 07/15] qmp-input-visitor: Favor new visit_free() function, Eric Blake, 2016/06/09
- [Qemu-devel] [PATCH v5 14/15] sockets: Use new QAPI cloning, Eric Blake, 2016/06/09
- [Qemu-devel] [PATCH v5 13/15] qapi: Add new clone visitor, Eric Blake, 2016/06/09
- [Qemu-devel] [PATCH v5 02/15] qemu-img: Don't leak errors when outputting JSON, Eric Blake, 2016/06/09
- [Qemu-devel] [PATCH v5 09/15] qmp-output-visitor: Favor new visit_free() function, Eric Blake, 2016/06/09
- [Qemu-devel] [PATCH v5 03/15] qapi: Add parameter to visit_end_*, Eric Blake, 2016/06/09
- [Qemu-devel] [PATCH v5 05/15] opts-visitor: Favor new visit_free() function, Eric Blake, 2016/06/09
- [Qemu-devel] [PATCH v5 12/15] qapi: Add new visit_complete() function, Eric Blake, 2016/06/09
- Re: [Qemu-devel] [PATCH v5 00/15] Add clone visitor,
Markus Armbruster <=