Re: [Qemu-devel] [PATCH v7 10/14] qapi: Detect collisions in C member names

2015-10-12 Thread Markus Armbruster
Eric Blake writes: > On 10/09/2015 08:11 AM, Markus Armbruster wrote: >> Eric Blake writes: >> >>> Detect attempts to declare two object members that would result >>> in the same C member name, by keying the 'seen' dictionary off >>> of the C name rather than the qapi name. It also requires pa

Re: [Qemu-devel] [PATCH v7 10/14] qapi: Detect collisions in C member names

2015-10-09 Thread Eric Blake
On 10/09/2015 08:11 AM, Markus Armbruster wrote: > Eric Blake writes: > >> Detect attempts to declare two object members that would result >> in the same C member name, by keying the 'seen' dictionary off >> of the C name rather than the qapi name. It also requires passing >> info through some o

Re: [Qemu-devel] [PATCH v7 10/14] qapi: Detect collisions in C member names

2015-10-09 Thread Markus Armbruster
Eric Blake writes: > Detect attempts to declare two object members that would result > in the same C member name, by keying the 'seen' dictionary off > of the C name rather than the qapi name. It also requires passing > info through some of the check() methods. > > This fixes two previously-brok

[Qemu-devel] [PATCH v7 10/14] qapi: Detect collisions in C member names

2015-10-03 Thread Eric Blake
Detect attempts to declare two object members that would result in the same C member name, by keying the 'seen' dictionary off of the C name rather than the qapi name. It also requires passing info through some of the check() methods. This fixes two previously-broken tests, and the resulting erro