Re: [Qemu-devel] [PATCH for-2.9 08/47] tests/qapi-schema: Cover empty union base

2017-03-15 Thread Markus Armbruster
Eric Blake writes: > On 03/13/2017 01:18 AM, Markus Armbruster wrote: >> The new test case shows off qapi.py choking on an empty union base. > > We're still finding ways to choke the parser ;) At least we're not finding dozens of ways without having to look hard anymore :) >> Signed-off-by: Mar

Re: [Qemu-devel] [PATCH for-2.9 08/47] tests/qapi-schema: Cover empty union base

2017-03-14 Thread Eric Blake
On 03/13/2017 01:18 AM, Markus Armbruster wrote: > The new test case shows off qapi.py choking on an empty union base. We're still finding ways to choke the parser ;) > > Signed-off-by: Markus Armbruster > --- Reviewed-by: Eric Blake -- Eric Blake eblake redhat com+1-919-301-3266 Libv

Re: [Qemu-devel] [PATCH for-2.9 08/47] tests/qapi-schema: Cover empty union base

2017-03-14 Thread Marc-André Lureau
On Mon, Mar 13, 2017 at 10:19 AM Markus Armbruster wrote: > The new test case shows off qapi.py choking on an empty union base. > > Signed-off-by: Markus Armbruster > Reviewed-by: Marc-André Lureau > --- > tests/Makefile.include | 1 + > tests/qapi-schema/union-base-empty.

[Qemu-devel] [PATCH for-2.9 08/47] tests/qapi-schema: Cover empty union base

2017-03-12 Thread Markus Armbruster
The new test case shows off qapi.py choking on an empty union base. Signed-off-by: Markus Armbruster --- tests/Makefile.include | 1 + tests/qapi-schema/union-base-empty.err | 10 ++ tests/qapi-schema/union-base-empty.exit | 1 + tests/qapi-schema/union-base-empty.jso