[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v7 07/11] iotests: add findtests.py
From: |
Kevin Wolf |
Subject: |
Re: [PATCH v7 07/11] iotests: add findtests.py |
Date: |
Fri, 22 Jan 2021 12:49:44 +0100 |
Am 16.01.2021 um 14:44 hat Vladimir Sementsov-Ogievskiy geschrieben:
> + if 'disabled' not in groups and 'disabled' not in exclude_groups:
> + # Don't want to modify function argument, so create new list.
> + exclude_groups = exclude_groups + ['disabled']
Oops, forgot the other comment I wanted to make:
This would only have been needed if you had turned exclude_groups into a
Sequence. Now that it's still a list, copying the list isn't strictly
necessary.
Kevin
- [PATCH v7 07/11] iotests: add findtests.py, (continued)
- [PATCH v7 07/11] iotests: add findtests.py, Vladimir Sementsov-Ogievskiy, 2021/01/16
- Re: [PATCH v7 07/11] iotests: add findtests.py, Eric Blake, 2021/01/21
- Re: [PATCH v7 07/11] iotests: add findtests.py, Eric Blake, 2021/01/21
- Re: [PATCH v7 07/11] iotests: add findtests.py, Kevin Wolf, 2021/01/22
- Re: [PATCH v7 07/11] iotests: add findtests.py, Vladimir Sementsov-Ogievskiy, 2021/01/22
- Re: [PATCH v7 07/11] iotests: add findtests.py, Kevin Wolf, 2021/01/22
- Re: [PATCH v7 07/11] iotests: add findtests.py, Vladimir Sementsov-Ogievskiy, 2021/01/22
- Re: [PATCH v7 07/11] iotests: add findtests.py, Kevin Wolf, 2021/01/22
- Re: [PATCH v7 07/11] iotests: add findtests.py, Vladimir Sementsov-Ogievskiy, 2021/01/22
Re: [PATCH v7 07/11] iotests: add findtests.py,
Kevin Wolf <=
[PATCH v7 09/11] iotests: add testrunner.py, Vladimir Sementsov-Ogievskiy, 2021/01/16
[PATCH v7 10/11] iotests: rewrite check into python, Vladimir Sementsov-Ogievskiy, 2021/01/16