[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 09/10] qmp.py: Avoid overriding a builtin obj
From: |
Eduardo Habkost |
Subject: |
Re: [Qemu-devel] [PATCH v2 09/10] qmp.py: Avoid overriding a builtin object |
Date: |
Tue, 25 Jul 2017 16:34:42 -0300 |
User-agent: |
Mutt/1.8.0 (2017-02-23) |
On Tue, Jul 25, 2017 at 05:09:50PM +0200, Lukáš Doktor wrote:
> The "id" is a builtin method to get object's identity and should not be
> overridden. This might bring some issues in case someone was directly
> calling "cmd(..., id=id)" but I haven't found such usage on brief search
> for "cmd\(.*id=".
>
> Signed-off-by: Lukáš Doktor <address@hidden>
I don't see the benefit of the patch, as the function is very
short and unlikely to ever use the id() builtin. But I am not
against it if it will silence code analyzer warnings.
> ---
> scripts/qmp/qmp.py | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/scripts/qmp/qmp.py b/scripts/qmp/qmp.py
> index f2f5a9b..ef12e8a 100644
> --- a/scripts/qmp/qmp.py
> +++ b/scripts/qmp/qmp.py
> @@ -177,19 +177,19 @@ class QEMUMonitorProtocol(object):
> print >>sys.stderr, "QMP:<<< %s" % resp
> return resp
>
> - def cmd(self, name, args=None, id=None):
> + def cmd(self, name, args=None, cmd_id=None):
> """
> Build a QMP command and send it to the QMP Monitor.
>
> @param name: command name (string)
> @param args: command arguments (dict)
> - @param id: command id (dict, list, string or int)
> + @param cmd_id: command id (dict, list, string or int)
> """
> qmp_cmd = {'execute': name}
> if args:
> qmp_cmd['arguments'] = args
> - if id:
> - qmp_cmd['id'] = id
> + if cmd_id:
> + qmp_cmd['id'] = cmd_id
> return self.cmd_obj(qmp_cmd)
>
> def command(self, cmd, **kwds):
> --
> 2.9.4
>
--
Eduardo
- [Qemu-devel] [PATCH v2 03/10] qemu.py: Use iteritems rather than keys(), (continued)
- [Qemu-devel] [PATCH v2 06/10] qmp.py: Couple of pylint/style fixes, Lukáš Doktor, 2017/07/25
- [Qemu-devel] [PATCH v2 07/10] qmp.py: Use object-based class for QEMUMonitorProtocol, Lukáš Doktor, 2017/07/25
- [Qemu-devel] [PATCH v2 08/10] qmp.py: Avoid "has_key" usage, Lukáš Doktor, 2017/07/25
- [Qemu-devel] [PATCH v2 09/10] qmp.py: Avoid overriding a builtin object, Lukáš Doktor, 2017/07/25
- Re: [Qemu-devel] [PATCH v2 09/10] qmp.py: Avoid overriding a builtin object,
Eduardo Habkost <=
[Qemu-devel] [PATCH v2 10/10] qtest.py: Few pylint/style fixes, Lukáš Doktor, 2017/07/25