[PATCH 0/5] qapi: Another round of minor fixes and cleanups

Markus Armbruster posted 5 patches 2 years, 7 months ago
Test checkpatch passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20210908045428.2689093-1-armbru@redhat.com
Maintainers: Michael Roth <michael.roth@amd.com>, Markus Armbruster <armbru@redhat.com>
scripts/qapi/common.py            | 19 ++++++-------------
scripts/qapi/expr.py              | 21 +++++++++++++--------
tests/qapi-schema/bad-if-not.err  |  2 ++
tests/qapi-schema/bad-if-not.json |  3 +++
tests/qapi-schema/bad-if-not.out  |  0
tests/qapi-schema/meson.build     |  1 +
6 files changed, 25 insertions(+), 21 deletions(-)
create mode 100644 tests/qapi-schema/bad-if-not.err
create mode 100644 tests/qapi-schema/bad-if-not.json
create mode 100644 tests/qapi-schema/bad-if-not.out
[PATCH 0/5] qapi: Another round of minor fixes and cleanups
Posted by Markus Armbruster 2 years, 7 months ago
Markus Armbruster (5):
  qapi: Fix a botched type annotation
  qapi: Drop Indentation.__bool__()
  qapi: Bury some unused code in class Indentation
  tests/qapi-schema: Cover 'not' condition with empty argument
  qapi: Fix bogus error for 'if': { 'not': '' }

 scripts/qapi/common.py            | 19 ++++++-------------
 scripts/qapi/expr.py              | 21 +++++++++++++--------
 tests/qapi-schema/bad-if-not.err  |  2 ++
 tests/qapi-schema/bad-if-not.json |  3 +++
 tests/qapi-schema/bad-if-not.out  |  0
 tests/qapi-schema/meson.build     |  1 +
 6 files changed, 25 insertions(+), 21 deletions(-)
 create mode 100644 tests/qapi-schema/bad-if-not.err
 create mode 100644 tests/qapi-schema/bad-if-not.json
 create mode 100644 tests/qapi-schema/bad-if-not.out

-- 
2.31.1


Re: [PATCH 0/5] qapi: Another round of minor fixes and cleanups
Posted by Marc-André Lureau 2 years, 7 months ago
Hi

On Wed, Sep 8, 2021 at 8:54 AM Markus Armbruster <armbru@redhat.com> wrote:

> Markus Armbruster (5):
>   qapi: Fix a botched type annotation
>   qapi: Drop Indentation.__bool__()
>   qapi: Bury some unused code in class Indentation
>   tests/qapi-schema: Cover 'not' condition with empty argument
>   qapi: Fix bogus error for 'if': { 'not': '' }
>
>  scripts/qapi/common.py            | 19 ++++++-------------
>  scripts/qapi/expr.py              | 21 +++++++++++++--------
>  tests/qapi-schema/bad-if-not.err  |  2 ++
>  tests/qapi-schema/bad-if-not.json |  3 +++
>  tests/qapi-schema/bad-if-not.out  |  0
>  tests/qapi-schema/meson.build     |  1 +
>  6 files changed, 25 insertions(+), 21 deletions(-)
>  create mode 100644 tests/qapi-schema/bad-if-not.err
>  create mode 100644 tests/qapi-schema/bad-if-not.json
>  create mode 100644 tests/qapi-schema/bad-if-not.out
>
>
Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com>
Re: [PATCH 0/5] qapi: Another round of minor fixes and cleanups
Posted by Markus Armbruster 2 years, 7 months ago
Queued.