Bug 688993

Summary: Make comparisons to None (singleton) done with 'is' and 'is not'
Product: [Community] Virtualization Tools Reporter: Satoru SATOH <ssato>
Component: virtinstAssignee: Cole Robinson <crobinso>
Severity: low Docs Contact:
Priority: unspecified    
Version: unspecifiedCC: berrange, crobinso, jforbes, virt-maint
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-01-24 20:35:43 EST Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Description Satoru SATOH 2011-03-18 14:22:33 EDT
Description of problem: This is a trivial patch to make comparisons to
None done with 'is'/'is not' instead of '=='/'!=' on the advice in
pep8 (http://www.python.org/dev/peps/pep-0008/):

   - Comparisons to singletons like None should always be done with
     'is' or 'is not', never the equality operators.

Version-Release number of selected component (if applicable):
I made a pathc for git HEAD but I guess it should be applicable to 
the versions of python-virtinst in fedora-14/15.
Comment 1 Cole Robinson 2011-03-21 11:53:35 EDT
Moved to upstream virtinst tracker. I'd be interested in seeing the patch
Comment 2 Cole Robinson 2012-01-24 20:35:43 EST
Closing since no patch provided and not really something that needs tracking in bugzilla.