Re: [PATCH v2 22/26] qapi: Improve reporting of invalid 'if' further

[Eric Blake]

On 9/27/19 8:46 AM, Markus Armbruster wrote:
> check_if()'s errors don't point to the offending part of the
> expression.  For instance:
>
>      tests/qapi-schema/alternate-branch-if-invalid.json:2: 'if' condition ' ' 
> makes no sense
>
> Other check_FOO() do, with the help of a @source argument.  Make
> check_if() do that, too.  The example above improves to:
>
>      tests/qapi-schema/alternate-branch-if-invalid.json:2: 'if' condition ' ' 
> of 'data' member 'branch' makes no sense
>
> Signed-off-by: Markus Armbruster <address@hidden>
> ---
Reviewed-by: Eric Blake <address@hidden>

--
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3226
Virtualization:  qemu.org | libvirt.org