[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 11/11] qtest.py: Avoid using mutable list as def
From: |
Eduardo Habkost |
Subject: |
Re: [Qemu-devel] [PATCH 11/11] qtest.py: Avoid using mutable list as default argument |
Date: |
Thu, 20 Jul 2017 15:44:14 -0300 |
User-agent: |
Mutt/1.8.0 (2017-02-23) |
On Thu, Jul 20, 2017 at 06:28:15PM +0200, Lukáš Doktor wrote:
> The list is a mutable object and is dangerous to use it. Recently the
> QEMUMachine was adjusted to allow None as default, let's use it here as
> well.
>
> Signed-off-by: Lukáš Doktor <address@hidden>
This patch requires patch 02/11 and both are pretty small.
I suggest squashing them together.
--
Eduardo
- [Qemu-devel] [PATCH 00/11] qemu.py: Pylint/style fixes, Lukáš Doktor, 2017/07/20
- [Qemu-devel] [PATCH 04/11] qemu.py: Simplify QMP key-conversion, Lukáš Doktor, 2017/07/20
- [Qemu-devel] [PATCH 11/11] qtest.py: Avoid using mutable list as default argument, Lukáš Doktor, 2017/07/20
- Re: [Qemu-devel] [PATCH 11/11] qtest.py: Avoid using mutable list as default argument,
Eduardo Habkost <=
- [Qemu-devel] [PATCH 03/11] qemu.py: Use iteritems rather than keys(), Lukáš Doktor, 2017/07/20
- [Qemu-devel] [PATCH 02/11] qemu.py: Avoid dangerous arguments, Lukáš Doktor, 2017/07/20
- [Qemu-devel] [PATCH 08/11] qmp.py: Avoid "has_key" usage, Lukáš Doktor, 2017/07/20