On Tue, 2010-06-08 at 20:50 +0300, Michael Goldish wrote: > On 06/08/2010 11:02 AM, Marc Enthus wrote: > > 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 > > > > > > The condition 'if pa_type' is meant to exclude both None and "". Both > are possible and both indicate that the user isn't interested in pa_type > at all. In that case 'if pa_type' conforms to PEP 8 AFAIK. I'm not > sure about the autotest coding style though. Yes, turns out I have overlooked the possibility of pa_type being "". I'll revert this fix, thanks for pointing this out! -- 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