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
>