Re: [Qemu-devel] [PATCH v5 12/46] qapi: Track location that created an implicit type

2015-10-01 Thread Eric Blake
On 09/29/2015 02:02 AM, Markus Armbruster wrote: While at it, we were always passing None as the location of array types, making that parameter useless; sharing the location (if any) of the underlying element type makes sense. >>> >>> The parameter is useless only because all array

Re: [Qemu-devel] [PATCH v5 12/46] qapi: Track location that created an implicit type

2015-09-29 Thread Markus Armbruster
Eric Blake writes: > On 09/28/2015 06:56 AM, Markus Armbruster wrote: >> Eric Blake writes: >> >>> A future patch will enable error detection in the various >>> QapiSchema check() methods. But since all errors have to >>> have an associated 'info'

Re: [Qemu-devel] [PATCH v5 12/46] qapi: Track location that created an implicit type

2015-09-28 Thread Eric Blake
On 09/28/2015 06:56 AM, Markus Armbruster wrote: > Eric Blake writes: > >> A future patch will enable error detection in the various >> QapiSchema check() methods. But since all errors have to >> have an associated 'info' location, we need a location to >> be associated with

Re: [Qemu-devel] [PATCH v5 12/46] qapi: Track location that created an implicit type

2015-09-28 Thread Markus Armbruster
Eric Blake writes: > A future patch will enable error detection in the various > QapiSchema check() methods. But since all errors have to > have an associated 'info' location, we need a location to > be associated with all implicit types. Easiest is to reuse > the location

[Qemu-devel] [PATCH v5 12/46] qapi: Track location that created an implicit type

2015-09-21 Thread Eric Blake
A future patch will enable error detection in the various QapiSchema check() methods. But since all errors have to have an associated 'info' location, we need a location to be associated with all implicit types. Easiest is to reuse the location of the enclosing entity that includes the