[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 09/10] qga: process_event() simplification
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH v4 09/10] qga: process_event() simplification |
Date: |
Mon, 08 Oct 2018 15:25:01 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.1 (gnu/linux) |
Marc-André Lureau <address@hidden> writes:
> Hi
> On Thu, Aug 30, 2018 at 6:03 PM Markus Armbruster <address@hidden> wrote:
>>
>> Marc-André Lureau <address@hidden> writes:
>>
>> > Simplify the code around qmp_dispatch():
>> > - rely on qmp_dispatch/check_obj() for message checking
>> > - have a single send_response() point
>> > - constify send_response() argument
>> >
>> > It changes a couple of error messages:
>> >
>> > * When @req isn't a dictionary, from
>> > Invalid JSON syntax
>> > to
>> > QMP input must be a JSON object
>> >
>> > * When @req lacks member "execute", from
>> > this feature or command is not currently supported
>> > to
>> > QMP input lacks member 'execute'
>> >
>> > Signed-off-by: Marc-André Lureau <address@hidden>
>> > Reviewed-by: Michael Roth <address@hidden>
>>
>> Reviewed-by: Markus Armbruster <address@hidden>
>>
>
> Can someone take the patch? thanks
Michael?