From: Marc Enthus <cxzdes@xxxxxxxxx> According to autotest coding style and http://www.python.org/dev/peps/pep-0008/ use if pa_type is not None: instead of if pa_type: Signed-off-by: Marc Enthus <cxzdes@xxxxxxxxx> --- client/tests/kvm/kvm_vm.py | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/client/tests/kvm/kvm_vm.py b/client/tests/kvm/kvm_vm.py index f3ce4d6..0080b15 100755 --- a/client/tests/kvm/kvm_vm.py +++ b/client/tests/kvm/kvm_vm.py @@ -516,7 +516,7 @@ class VM: pa_type) return False - elif pa_type and pa_type != "no": + elif pa_type is not None and pa_type != "no": logging.warn("Unsupported pci_assignable type: %s", pa_type) # Make qemu command -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html