Re: [PATCH 2/7] qapi: Store pragma state in QAPISourceInfo, not global state

[Eric Blake]

On 10/1/19 2:15 PM, Markus Armbruster wrote:
> The frontend can't be run more than once due to its global state.
> A future commit will want to do that.
>
> Recent commit "qapi: Move context-sensitive checking to the proper
> place" got rid of many global variables already, but pragma state is
> still stored in global variables (that's why a pragma directive's
> scope is the complete schema).
>
> Move the pragma state to QAPISourceInfo.
>
> Signed-off-by: Markus Armbruster <address@hidden>
> ---
>   scripts/qapi/common.py | 36 +++++++++++++++++++-----------------
>   1 file changed, 19 insertions(+), 17 deletions(-)
Reviewed-by: Eric Blake <address@hidden>

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