[v1] qemu.py: Pylint/style fixes

[Qemu-devel] [PATCH 00/11] qemu.py: Pylint/style fixes

Lukáš Doktor posted 11 patches 6 years, 9 months ago

Diff against v2 v3 v4 v5 v6
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20170720162815.19802-1-ldoktor@redhat.com

Test FreeBSD passed

Test checkpatch passed

Test docker passed

Test s390x passed

There is a newer version of this series

scripts/qemu.py    | 98 +++++++++++++++++++++++++++++++++++++++---------------
scripts/qmp/qmp.py | 49 ++++++++++++++++-----------
scripts/qtest.py   | 14 ++++----
3 files changed, 108 insertions(+), 53 deletions(-)

Expand all Fold all

[Qemu-devel] [PATCH 00/11] qemu.py: Pylint/style fixes

Posted by Lukáš Doktor 6 years, 9 months ago

Hello guys,

I'm reading the available python modules to exercise qemu and while reading them
I fixed some issues that caught my attention. It usually starts with a simple
pylint/docstring fixes and slowly graduates to more controversial ones so I'm
open to suggestion to remove some of them.

Kind regards,
Lukáš

Lukáš Doktor (11):
  qemu.py: Pylint/style fixes
  qemu.py: Avoid dangerous arguments
  qemu.py: Use iteritems rather than keys()
  qemu.py: Simplify QMP key-conversion
  qemu.py: Use custom exceptions rather than Exception
  qmp.py: Couple of pylint/style fixes
  qmp.py: Use object-based class for QEMUMonitorProtocol
  qmp.py: Avoid "has_key" usage
  qmp.py: Avoid overriding a builtin object
  qtest.py: Few pylint/style fixes
  qtest.py: Avoid using mutable list as default argument

 scripts/qemu.py    | 98 +++++++++++++++++++++++++++++++++++++++---------------
 scripts/qmp/qmp.py | 49 ++++++++++++++++-----------
 scripts/qtest.py   | 14 ++++----
 3 files changed, 108 insertions(+), 53 deletions(-)

-- 
2.9.4