On 3/4/20 10:59 AM, Markus Armbruster wrote:
> Signed-off-by: Markus Armbruster <arm...@redhat.com>
> ---
>  scripts/qapi/commands.py   |  2 +-
>  scripts/qapi/expr.py       |  3 +--
>  scripts/qapi/gen.py        |  9 ++++++---
>  scripts/qapi/introspect.py |  2 --
>  scripts/qapi/parser.py     |  6 ++----
>  scripts/qapi/schema.py     | 11 +++++------
>  6 files changed, 15 insertions(+), 18 deletions(-)
> 

Looks okay. (I don't care as much about no-else-return being there or
not, and this module is your baby.)

Reviewed-by: John Snow <js...@redhat.com>


Reply via email to