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 <arm...@redhat.com>
---
Reviewed-by: Eric Blake <ebl...@redhat.com>

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

Reply via email to