Re: [Qemu-devel] [PATCH v8 28/40] qapi: Prefer 'struct' over 'type' in generator

[Markus Armbruster]

Eric Blake <address@hidden> writes:

> Referring to "type" as both a meta-type (built-in, enum, union,
> alternate, or struct) and a specific type (the name that the
> schema uses for declaring structs) is confusing.  The confusion
> is only made worse by the fact that the generator mostly already
> refers to struct even when dealing with expr['type'].  This
> commit changes the generator to consistently refer to it as
> struct everywhere, plus a single back-compat tweak that allows
> accepting the existing .json files as-is, so that the meat of
> this change is separate from the mindless churn of that change.
>
> Fix the testsuite fallout for error messages that change, and
> in some cases, become more legible.  Improve comments to better
> match our intentions where a struct (rather than any complex
> type) is required.  Note that in some cases, an error message
> now refers to 'struct' while the schema still refers to 'type';
> that will be cleaned up in the later commit to the schema.
>
> Signed-off-by: Eric Blake <address@hidden>

Reviewed-by: Markus Armbruster <address@hidden>