Re: [Qemu-devel] [PATCH RFC v2 45/47] qapi: New QMP command query-schema for QMP schema introspection

2015-08-03 Thread Markus Armbruster
Markus Armbruster arm...@redhat.com writes: Eric Blake ebl...@redhat.com writes: On 07/29/2015 03:19 AM, Markus Armbruster wrote: Longest line is a bit over 4KiB for me. If we break up string literals, at least use some indentation to make it obvious that multiple lines merge to a single

Re: [Qemu-devel] [PATCH RFC v2 45/47] qapi: New QMP command query-schema for QMP schema introspection

2015-07-29 Thread Markus Armbruster
Eric Blake ebl...@redhat.com writes: On 07/29/2015 03:19 AM, Markus Armbruster wrote: Longest line is a bit over 4KiB for me. If we break up string literals, at least use some indentation to make it obvious that multiple lines merge to a single array entry. For example (after patch 47):

Re: [Qemu-devel] [PATCH RFC v2 45/47] qapi: New QMP command query-schema for QMP schema introspection

2015-07-29 Thread Eric Blake
On 07/29/2015 03:19 AM, Markus Armbruster wrote: Longest line is a bit over 4KiB for me. If we break up string literals, at least use some indentation to make it obvious that multiple lines merge to a single array entry. For example (after patch 47): ... { 'name': ':abr', 'meta-type':

Re: [Qemu-devel] [PATCH RFC v2 45/47] qapi: New QMP command query-schema for QMP schema introspection

2015-07-29 Thread Markus Armbruster
Eric Blake ebl...@redhat.com writes: On 07/28/2015 08:33 AM, Markus Armbruster wrote: Eric Blake ebl...@redhat.com writes: On 07/01/2015 02:22 PM, Markus Armbruster wrote: Caution, rough edges. No joke. It doesn't even compile without this fixup to a rebase snafu (see [0] below): Uh,

Re: [Qemu-devel] [PATCH RFC v2 45/47] qapi: New QMP command query-schema for QMP schema introspection

2015-07-28 Thread Markus Armbruster
Eric Blake ebl...@redhat.com writes: On 07/01/2015 02:22 PM, Markus Armbruster wrote: Caution, rough edges. No joke. It doesn't even compile without this fixup to a rebase snafu (see [0] below): Uh, how did that happen? I compiled it a million times... (except for the last time,

Re: [Qemu-devel] [PATCH RFC v2 45/47] qapi: New QMP command query-schema for QMP schema introspection

2015-07-28 Thread Eric Blake
On 07/28/2015 08:33 AM, Markus Armbruster wrote: Eric Blake ebl...@redhat.com writes: On 07/01/2015 02:22 PM, Markus Armbruster wrote: Caution, rough edges. No joke. It doesn't even compile without this fixup to a rebase snafu (see [0] below): Uh, how did that happen? I compiled it a

Re: [Qemu-devel] [PATCH RFC v2 45/47] qapi: New QMP command query-schema for QMP schema introspection

2015-07-23 Thread Eric Blake
On 07/01/2015 02:22 PM, Markus Armbruster wrote: Caution, rough edges. No joke. It doesn't even compile without this fixup to a rebase snafu (see [0] below): diff --git i/scripts/qapi-types.py w/scripts/qapi-types.py index 79e8d24..12f3767 100644 --- i/scripts/qapi-types.py +++

[Qemu-devel] [PATCH RFC v2 45/47] qapi: New QMP command query-schema for QMP schema introspection

2015-07-01 Thread Markus Armbruster
Caution, rough edges. qapi/introspect.json defines the introspection schema. It should do for uses other than QMP. FIXME it's almost entirely devoid of comments. The introspection schema does not reflect all the rules and restrictions that apply to QAPI schemata. A valid QAPI schema has an