[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 1/2] tests/qapi-schema: Use Python OSError instead of outmode
From: |
Markus Armbruster |
Subject: |
Re: [PATCH 1/2] tests/qapi-schema: Use Python OSError instead of outmoded IOError |
Date: |
Thu, 23 Sep 2021 11:33:39 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/27.2 (gnu/linux) |
John Snow <jsnow@redhat.com> writes:
> On Wed, Sep 22, 2021 at 8:56 AM Markus Armbruster <armbru@redhat.com> wrote:
>
>> Signed-off-by: Markus Armbruster <armbru@redhat.com>
>> ---
>> tests/qapi-schema/test-qapi.py | 4 ++--
>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/tests/qapi-schema/test-qapi.py
>> b/tests/qapi-schema/test-qapi.py
>> index 73cffae2b6..2e384f5efd 100755
>> --- a/tests/qapi-schema/test-qapi.py
>> +++ b/tests/qapi-schema/test-qapi.py
>> @@ -154,7 +154,7 @@ def test_and_diff(test_name, dir_name, update):
>> errfp = open(os.path.join(dir_name, test_name + '.err'), mode)
>> expected_out = outfp.readlines()
>> expected_err = errfp.readlines()
>> - except IOError as err:
>> + except OSError as err:
>> print("%s: can't open '%s': %s"
>> % (sys.argv[0], err.filename, err.strerror),
>> file=sys.stderr)
>> @@ -180,7 +180,7 @@ def test_and_diff(test_name, dir_name, update):
>> errfp.truncate(0)
>> errfp.seek(0)
>> errfp.writelines(actual_err)
>> - except IOError as err:
>> + except OSError as err:
>> print("%s: can't write '%s': %s"
>> % (sys.argv[0], err.filename, err.strerror),
>> file=sys.stderr)
>> --
>> 2.31.1
>>
>>
> If you're happy with the expanded scope of the exception-catcher, I am too.
https://docs.python.org/3.6/library/exceptions.html has
Changed in version 3.3: EnvironmentError, IOError, WindowsError,
socket.error, select.error and mmap.error have been merged into
OSError, and the constructor may return a subclass.
and
The following exceptions are kept for compatibility with previous
versions; starting from Python 3.3, they are aliases of OSError.
exception EnvironmentError
exception IOError
exception WindowsError
Only available on Windows.
So unless I'm misunderstanding something (which is quite possible),
we're catching exactly the same exceptions as before, we just switch to
their preferred name.
> Reviewed-by: John Snow <jsnow@redhat.com>
Thanks!