On Wed, Oct 25, 2023 at 5:36 AM Markus Armbruster <arm...@redhat.com> wrote: > > Pylint warns: > > tests/qapi-schema/test-qapi.py:139:13: W1514: Using open without > explicitly specifying an encoding (unspecified-encoding) > tests/qapi-schema/test-qapi.py:143:13: W1514: Using open without > explicitly specifying an encoding (unspecified-encoding) > > Add encoding='utf-8'.
This one I think actually matters in some cases. Some esoteric cases where the platform default isn't UTF-8. Might happen on Windows where it wants to use UTF-16 or UCS-2 or whatever. > > Pylint advises: > > tests/qapi-schema/test-qapi.py:143:13: R1732: Consider using 'with' for > resource-allocating operations (consider-using-with) > > Silence this by returning the value directly. Smart! > > Pylint advises: > > tests/qapi-schema/test-qapi.py:221:4: R1722: Consider using sys.exit() > (consider-using-sys-exit) > tests/qapi-schema/test-qapi.py:226:4: R1722: Consider using sys.exit() > (consider-using-sys-exit) > > Sure, why not. I have no idea why this matters, but I habitually use the sys.exit() form ... https://docs.python.org/3.11/library/constants.html#constants-added-by-the-site-module Oh, that's weird. Well, Sure, why not. > > Signed-off-by: Markus Armbruster <arm...@redhat.com> Reviewed-by: John Snow <js...@redhat.com> > --- > tests/qapi-schema/test-qapi.py | 9 ++++----- > 1 file changed, 4 insertions(+), 5 deletions(-) > > diff --git a/tests/qapi-schema/test-qapi.py b/tests/qapi-schema/test-qapi.py > index d58c31f539..14f7b62a44 100755 > --- a/tests/qapi-schema/test-qapi.py > +++ b/tests/qapi-schema/test-qapi.py > @@ -136,12 +136,11 @@ def test_frontend(fname): > def open_test_result(dir_name, file_name, update): > mode = 'r+' if update else 'r' > try: > - fp = open(os.path.join(dir_name, file_name), mode) > + return open(os.path.join(dir_name, file_name), mode, > encoding='utf-8') > except FileNotFoundError: > if not update: > raise > - fp = open(os.path.join(dir_name, file_name), 'w+') > - return fp > + return open(os.path.join(dir_name, file_name), 'w+', encoding='utf-8') > > > def test_and_diff(test_name, dir_name, update): > @@ -218,9 +217,9 @@ def main(argv): > test_name = os.path.splitext(base_name)[0] > status |= test_and_diff(test_name, dir_name, args.update) > > - exit(status) > + sys.exit(status) > > > if __name__ == '__main__': > main(sys.argv) > - exit(0) > + sys.exit(0) > -- > 2.41.0 > >
