Message ID | 20200922223525.4085762-1-jsnow@redhat.com |
---|---|
Headers | show |
Series | qapi: static typing conversion, pt5 | expand |
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 >