Re: [Qemu-devel] [PATCH v2 15/18] iotests: teach FilePath to produce multiple paths

[Max Reitz]

On 03.07.19 23:55, John Snow wrote:
> Use "FilePaths" instead of "FilePath" to request multiple files be
> cleaned up after we leave that object's scope.
> 
> This is not crucial; but it saves a little typing.
> 
> Signed-off-by: John Snow <address@hidden>
> ---
>  tests/qemu-iotests/iotests.py | 33 ++++++++++++++++++++++-----------
>  1 file changed, 22 insertions(+), 11 deletions(-)
> 
> diff --git a/tests/qemu-iotests/iotests.py b/tests/qemu-iotests/iotests.py
> index c544659ecb..db00fd2adf 100644
> --- a/tests/qemu-iotests/iotests.py
> +++ b/tests/qemu-iotests/iotests.py

[...]

> @@ -391,7 +403,6 @@ def file_path_remover():
>          except OSError:
>              pass
>  
> -
>  def file_path(*names):
>      ''' Another way to get auto-generated filename that cleans itself up.
>  

Not sure about this hunk, but OK.

Reviewed-by: Max Reitz <address@hidden>

signature.asc
Description: OpenPGP digital signature