mbox series

[00/26] qapi: static typing conversion, pt5

Message ID 20200922223525.4085762-1-jsnow@redhat.com
Headers show
Series qapi: static typing conversion, pt5 | expand

Message

John Snow Sept. 22, 2020, 10:34 p.m. UTC
based-on: <20200922210101.4081073-1-jsnow@redhat.com>
          [PATCH 0/6] qapi: static typing conversion, pt4

Hi, this series adds static type hints to the QAPI module.
This is part five!

Part 5: https://gitlab.com/jsnow/qemu/-/tree/python-qapi-cleanup-pt5
Everything: https://gitlab.com/jsnow/qemu/-/tree/python-qapi-cleanup-pt6

- Requires Python 3.6+
- Requires mypy 0.770 or newer (for type analysis only)
- Requires pylint 2.6.0 or newer (for lint checking only)

This part of the series focuses on just parser.py.

Type hints are added in patches that add *only* type hints and change no
other behavior. Any necessary changes to behavior to accommodate typing
are split out into their own tiny patches.

Every commit should pass with:
 - flake8 qapi/
 - pylint --rcfile=qapi/pylintrc qapi/
 - mypy --config-file=qapi/mypy.ini qapi/

John Snow (26):
  qapi/parser.py: refactor parsing routine into method
  qapi/parser.py: group variable declarations in __init__
  qapi/parser.py: use 'with' statement for opening files
  qapi/source.py: Add default arguments to QAPISourceInfo
  qapi/parser.py: start source info at line 0
  qapi/parser.py: raise QAPIParseError during file opening
  qapi/parser.py: fully remove 'null' constant
  qapi/parser.py: Assert lexer value is a string
  qapi/parser.py: assert get_expr returns object in outer loop
  qapi/parser.py: assert object keys are strings
  qapi/parser.py: Convert several methods to @classmethod
  qapi/parser.py: add casts to pragma checks
  qapi/parser.py: add type hint annotations
  qapi/parser.py: add docstrings
  qapi/parser.py: add ParsedExpression type
  qapi/pragma.py: Move QAPISchemaPragma into its own module
  qapi/pragma.py: Move pragma parsing out of parser.py
  qapi/parser.py: Modify _include() to use parser state
  qapi/parser.py: add parent argument
  qapi/parser.py: remove unused check_args_section arguments
  qapi/parser.py: QAPIDoc: convert @staticmethod to @classmethod
  qapi/parser.py: add type hint annotations (QAPIDoc)
  qapi/parser.py: enable mypy checks
  qapi/parser.py: remove one and two-letter variables
  qapi/parser.py: Silence too-few-public-methods warning
  qapi/parser.py: enable pylint checks

 scripts/qapi/error.py                     |   8 +-
 scripts/qapi/expr.py                      |  56 ++--
 scripts/qapi/mypy.ini                     |   5 -
 scripts/qapi/parser.py                    | 322 +++++++++++++---------
 scripts/qapi/pragma.py                    |  68 +++++
 scripts/qapi/pylintrc                     |   3 +-
 scripts/qapi/schema.py                    |   6 +-
 scripts/qapi/source.py                    |  22 +-
 tests/qapi-schema/leading-comma-list.err  |   2 +-
 tests/qapi-schema/trailing-comma-list.err |   2 +-
 10 files changed, 301 insertions(+), 193 deletions(-)
 create mode 100644 scripts/qapi/pragma.py

-- 
2.26.2

Comments

John Snow Sept. 22, 2020, 10:54 p.m. UTC | #1
On 9/22/20 6:34 PM, John Snow wrote:
> based-on: <20200922210101.4081073-1-jsnow@redhat.com>

^ Copy-paste malfunction

based-on: <20200922212115.4084301-1-jsnow@redhat.com>

>            [PATCH 0/6] qapi: static typing conversion, pt4
> 
> Hi, this series adds static type hints to the QAPI module.
> This is part five!
> 
> Part 5: https://gitlab.com/jsnow/qemu/-/tree/python-qapi-cleanup-pt5
> Everything: https://gitlab.com/jsnow/qemu/-/tree/python-qapi-cleanup-pt6
> 
> - Requires Python 3.6+
> - Requires mypy 0.770 or newer (for type analysis only)
> - Requires pylint 2.6.0 or newer (for lint checking only)
> 
> This part of the series focuses on just parser.py.
> 
> Type hints are added in patches that add *only* type hints and change no
> other behavior. Any necessary changes to behavior to accommodate typing
> are split out into their own tiny patches.
> 
> Every commit should pass with:
>   - flake8 qapi/
>   - pylint --rcfile=qapi/pylintrc qapi/
>   - mypy --config-file=qapi/mypy.ini qapi/
> 
> John Snow (26):
>    qapi/parser.py: refactor parsing routine into method
>    qapi/parser.py: group variable declarations in __init__
>    qapi/parser.py: use 'with' statement for opening files
>    qapi/source.py: Add default arguments to QAPISourceInfo
>    qapi/parser.py: start source info at line 0
>    qapi/parser.py: raise QAPIParseError during file opening
>    qapi/parser.py: fully remove 'null' constant
>    qapi/parser.py: Assert lexer value is a string
>    qapi/parser.py: assert get_expr returns object in outer loop
>    qapi/parser.py: assert object keys are strings
>    qapi/parser.py: Convert several methods to @classmethod
>    qapi/parser.py: add casts to pragma checks
>    qapi/parser.py: add type hint annotations
>    qapi/parser.py: add docstrings
>    qapi/parser.py: add ParsedExpression type
>    qapi/pragma.py: Move QAPISchemaPragma into its own module
>    qapi/pragma.py: Move pragma parsing out of parser.py
>    qapi/parser.py: Modify _include() to use parser state
>    qapi/parser.py: add parent argument
>    qapi/parser.py: remove unused check_args_section arguments
>    qapi/parser.py: QAPIDoc: convert @staticmethod to @classmethod
>    qapi/parser.py: add type hint annotations (QAPIDoc)
>    qapi/parser.py: enable mypy checks
>    qapi/parser.py: remove one and two-letter variables
>    qapi/parser.py: Silence too-few-public-methods warning
>    qapi/parser.py: enable pylint checks
> 
>   scripts/qapi/error.py                     |   8 +-
>   scripts/qapi/expr.py                      |  56 ++--
>   scripts/qapi/mypy.ini                     |   5 -
>   scripts/qapi/parser.py                    | 322 +++++++++++++---------
>   scripts/qapi/pragma.py                    |  68 +++++
>   scripts/qapi/pylintrc                     |   3 +-
>   scripts/qapi/schema.py                    |   6 +-
>   scripts/qapi/source.py                    |  22 +-
>   tests/qapi-schema/leading-comma-list.err  |   2 +-
>   tests/qapi-schema/trailing-comma-list.err |   2 +-
>   10 files changed, 301 insertions(+), 193 deletions(-)
>   create mode 100644 scripts/qapi/pragma.py
>