diff options
| author | Johannes Erdfelt <johannes.erdfelt@rackspace.com> | 2012-06-08 17:08:14 +0000 |
|---|---|---|
| committer | Johannes Erdfelt <johannes.erdfelt@rackspace.com> | 2012-06-08 17:08:14 +0000 |
| commit | fb0f00e946eff84e855d0721830b5016fe80ea8d (patch) | |
| tree | c83af27ac8919db9aa77e24ef752f395dd2086a1 /openstack | |
| parent | 2d9463709588d88172e125a97200ac813ee79eb4 (diff) | |
| download | oslo-fb0f00e946eff84e855d0721830b5016fe80ea8d.tar.gz oslo-fb0f00e946eff84e855d0721830b5016fe80ea8d.tar.xz oslo-fb0f00e946eff84e855d0721830b5016fe80ea8d.zip | |
Use 'is not None' instead of '!= None'
Fixes bug 1010570
pep8 suggests the former over the latter
Change-Id: Ice3a3b1cc2eea9228fffb4ee40fc360ff79054a3
Diffstat (limited to 'openstack')
| -rw-r--r-- | openstack/common/cfg.py | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/openstack/common/cfg.py b/openstack/common/cfg.py index 5c32543..f97f718 100644 --- a/openstack/common/cfg.py +++ b/openstack/common/cfg.py @@ -522,7 +522,7 @@ class Opt(object): self.help = help self.secret = secret self.required = required - if deprecated_name != None: + if deprecated_name is not None: self.deprecated_name = deprecated_name.replace('-', '_') else: self.deprecated_name = None @@ -541,7 +541,7 @@ class Opt(object): def _cparser_get_with_deprecated(self, cparser, section): """If cannot find option as dest try deprecated_name alias.""" - if self.deprecated_name != None: + if self.deprecated_name is not None: return cparser.get(section, [self.dest, self.deprecated_name]) return cparser.get(section, [self.dest]) @@ -755,7 +755,7 @@ class MultiStrOpt(Opt): def _cparser_get_with_deprecated(self, cparser, section): """If cannot find option as dest try deprecated_name alias.""" - if self.deprecated_name != None: + if self.deprecated_name is not None: return cparser.get(section, [self.dest, self.deprecated_name], multi=True) return cparser.get(section, [self.dest], multi=True) |
