[Top][All Lists]

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

Re: [Qemu-devel] [PATCH 0/2] migration: Move qjson.[ch] to migration/, c

From: Markus Armbruster
Subject: Re: [Qemu-devel] [PATCH 0/2] migration: Move qjson.[ch] to migration/, clean up
Date: Fri, 06 May 2016 15:11:04 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux)

Eric Blake <address@hidden> writes:

> On 05/04/2016 10:49 AM, Markus Armbruster wrote:
>> qjson.[ch] is a JSON writer used by migration.  Eric proposed to
>> replace it by common code in his "Add qapi-to-JSON and clone visitors"
>> series.  David's review led to the conclusion that migration would
>> prefer to keep its own JSON writer, to better serve its requirements.
>> PATCH 1 move it to its proper place, and explains why it exists in a
>> bit more detail.  PATCH 2 simplifies it a bit.
> Whose tree would this go in through?

Migration is the natural choice.  Would that inconvenience you?

> At any rate, series:
> Reviewed-by: Eric Blake <address@hidden>
> but see nit on 1/2

Perhaps the maintainer can add the commit hashes on merge.


reply via email to

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