[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/3] qemu.py: Use items() instead of iteritems()
From: |
Eduardo Habkost |
Subject: |
Re: [Qemu-devel] [PATCH 1/3] qemu.py: Use items() instead of iteritems() |
Date: |
Mon, 12 Mar 2018 22:25:15 -0300 |
User-agent: |
Mutt/1.9.2 (2017-12-15) |
On Tue, Mar 13, 2018 at 12:02:39AM +0100, Philippe Mathieu-Daudé wrote:
> On 03/12/2018 07:55 PM, Eduardo Habkost wrote:
> > items() is less efficient on Python 2.x, but makes the code work
> > on both Python 2 and Python 3.
> >
> > Cc: Lukáš Doktor <address@hidden>
> > Cc: Philippe Mathieu-Daudé <address@hidden>
> > Cc: Cleber Rosa <address@hidden>
> > Signed-off-by: Eduardo Habkost <address@hidden>
> > ---
> > scripts/qemu.py | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/scripts/qemu.py b/scripts/qemu.py
> > index 305a946562..08a3e9af5a 100644
> > --- a/scripts/qemu.py
> > +++ b/scripts/qemu.py
> > @@ -277,7 +277,7 @@ class QEMUMachine(object):
> > def qmp(self, cmd, conv_keys=True, **args):
> > '''Invoke a QMP command and return the response dict'''
> > qmp_args = dict()
> > - for key, value in args.iteritems():
> > + for key, value in args.items():
> > if conv_keys:
> > qmp_args[key.replace('_', '-')] = value
> > else:
> >
>
> http://lists.nongnu.org/archive/html/qemu-devel/2017-12/msg04046.html
Oops, that series had fallen through the cracks, sorry!
I remember merging a series with fixes for Python 3, but now I
see there were two separate series: "Support building with py2 or
py3¨ (already merged), and yours ("iotests: python3
compatibility¨).
--
Eduardo