[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 43/50] build-sys: move qmp-introspect per tar
From: |
Marc-André Lureau |
Subject: |
Re: [Qemu-devel] [PATCH v3 43/50] build-sys: move qmp-introspect per target |
Date: |
Thu, 11 Jan 2018 22:32:17 +0100 |
Hi
On Thu, Dec 14, 2017 at 5:30 PM, Markus Armbruster <address@hidden> wrote:
> Marc-André Lureau <address@hidden> writes:
>
>> The following patches are going to introduce per-target #ifdef, and
>> but the introspection data is generated only once, and must thus be
>> built with the target.
>
> "and but"?
>
>> Drop "do_test_visitor_in_qmp_introspect(&&qmp_schema_qlit)" since it
>
> s/&&/&/
>
>> is no longer in a common object, and covered by "query-qmp-schema
>> test" instead.
>
> Do you mean test /x86_64/qmp/query-qmp-schema of tests/qmp-test.c?
>
I updated the commit message.
> That test tests significantly less. It runs the command, tests it
> succeeds and returns something. The test you remove additionally tests
> the returned value conforms to the schema, by passing it to
> visit_type_SchemaInfoList().
>
> Perhaps we can enhance the test you keep to avoid this loss.
>
It does call visit_type_SchemaInfoList() already on the return value.
thanks
--
Marc-André Lureau
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-devel] [PATCH v3 43/50] build-sys: move qmp-introspect per target,
Marc-André Lureau <=