diff options
| author | lzyeval <lzyeval@gmail.com> | 2012-02-02 23:24:46 +0000 |
|---|---|---|
| committer | Mark McLoughlin <markmc@redhat.com> | 2012-02-02 23:27:11 +0000 |
| commit | 8441babf22a597de4a526ec11538cb0c6b503646 (patch) | |
| tree | 746fdd15418d7b6b2423b24bef480bba7c1226c9 /openstack | |
| parent | af365e448b805812e0cec527f97373e8a050d64a (diff) | |
| download | oslo-8441babf22a597de4a526ec11538cb0c6b503646.tar.gz oslo-8441babf22a597de4a526ec11538cb0c6b503646.tar.xz oslo-8441babf22a597de4a526ec11538cb0c6b503646.zip | |
PEP8 type comparison cleanup
Fixes bug #910295
The None, True, and False values are singletons.
All variable *comparisons* to singletons should use 'is' or 'is not'.
All variable *evaluations* to boolean should use 'if' or 'if not'.
"== None", "== True", "== False", and "!= None" comparisons in sqlalchemy's
where(), or_(), filter(), and_(), and select() functions should not be changed.
Incorrect comparisons or evaluations in comments were not changed.
Change-Id: I087f0883bf115b5fe714ccfda86a794b9b2a87f7
Diffstat (limited to 'openstack')
| -rw-r--r-- | openstack/common/cfg.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/openstack/common/cfg.py b/openstack/common/cfg.py index 6070100..c8c6a52 100644 --- a/openstack/common/cfg.py +++ b/openstack/common/cfg.py @@ -850,7 +850,7 @@ class ConfigOpts(collections.Mapping, object): :return: False if the opt was already register, True otherwise :raises: DuplicateOptError, ArgsAlreadyParsedError """ - if self._args != None: + if self._args is not None: raise ArgsAlreadyParsedError("cannot register CLI option") if not self.register_opt(opt, group): |