From fb0f00e946eff84e855d0721830b5016fe80ea8d Mon Sep 17 00:00:00 2001 From: Johannes Erdfelt Date: Fri, 8 Jun 2012 17:08:14 +0000 Subject: Use 'is not None' instead of '!= None' Fixes bug 1010570 pep8 suggests the former over the latter Change-Id: Ice3a3b1cc2eea9228fffb4ee40fc360ff79054a3 --- openstack/common/cfg.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'openstack/common') 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) -- cgit