[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 09/11] qmp.py: Avoid overriding a builtin object
From: |
Eduardo Habkost |
Subject: |
Re: [Qemu-devel] [PATCH 09/11] qmp.py: Avoid overriding a builtin object |
Date: |
Thu, 20 Jul 2017 15:38:32 -0300 |
User-agent: |
Mutt/1.8.0 (2017-02-23) |
On Thu, Jul 20, 2017 at 06:28:13PM +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>
> ---
> 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 a14b001..c3e0206 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['cmd_id'] = cmd_id
The member sent through the monitor should still be called "id".
i.e.:
qmp_cmd['id'] = cmd_id
> return self.cmd_obj(qmp_cmd)
>
> def command(self, cmd, **kwds):
> --
> 2.9.4
>
--
Eduardo
- Re: [Qemu-devel] [PATCH 02/11] qemu.py: Avoid dangerous arguments, (continued)
[Qemu-devel] [PATCH 09/11] qmp.py: Avoid overriding a builtin object, Lukáš Doktor, 2017/07/20
- Re: [Qemu-devel] [PATCH 09/11] qmp.py: Avoid overriding a builtin object,
Eduardo Habkost <=
[Qemu-devel] [PATCH 10/11] qtest.py: Few pylint/style fixes, Lukáš Doktor, 2017/07/20
[Qemu-devel] [PATCH 01/11] qemu.py: Pylint/style fixes, Lukáš Doktor, 2017/07/20