Re: [PATCH v8 04/11] iotests: replace mutable list default args

2020-03-24 Thread Max Reitz
On 17.03.20 01:40, John Snow wrote: > It's bad hygiene: if we modify this list, it will be modified across all > invocations. > > (Remaining bad usages are fixed in a subsequent patch which changes the > function signature anyway.) > > Signed-off-by: John Snow > Reviewed-by: Philippe

[PATCH v8 04/11] iotests: replace mutable list default args

2020-03-16 Thread John Snow
It's bad hygiene: if we modify this list, it will be modified across all invocations. (Remaining bad usages are fixed in a subsequent patch which changes the function signature anyway.) Signed-off-by: John Snow Reviewed-by: Philippe Mathieu-Daudé --- tests/qemu-iotests/iotests.py | 24