diff options
| author | Ben Nemec <openstack@nemebean.com> | 2013-07-19 10:15:03 -0500 |
|---|---|---|
| committer | Ben Nemec <openstack@nemebean.com> | 2013-07-19 10:17:01 -0500 |
| commit | 05f33ec591acd2870130a2e5eb4c29994b11fbe6 (patch) | |
| tree | 4e13a901f962eb0816d98082a618c7aafafb045f /openstack/common | |
| parent | 9a3ebf2ff5b995085d0fb8e075687b48681d391f (diff) | |
| download | oslo-05f33ec591acd2870130a2e5eb4c29994b11fbe6.tar.gz oslo-05f33ec591acd2870130a2e5eb4c29994b11fbe6.tar.xz oslo-05f33ec591acd2870130a2e5eb4c29994b11fbe6.zip | |
Exception code cleanup
Remove old commented code
Remove some incorrect comments
Remove unnecessary __class__ indirection
Convert err.__str__() to str(err)
Remove unused tb item in test
Change-Id: I9adaa90bbd6dffd7f58f5ff0403859e8806e37fb
Diffstat (limited to 'openstack/common')
| -rw-r--r-- | openstack/common/exception.py | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/openstack/common/exception.py b/openstack/common/exception.py index 2a9a751..13c3bff 100644 --- a/openstack/common/exception.py +++ b/openstack/common/exception.py @@ -47,7 +47,7 @@ class UnknownScheme(Error): msg_fmt = "Unknown scheme '%s' found in URI" def __init__(self, scheme): - msg = self.__class__.msg_fmt % scheme + msg = self.msg_fmt % scheme super(UnknownScheme, self).__init__(msg) @@ -56,7 +56,7 @@ class BadStoreUri(Error): msg_fmt = "The Store URI %s was malformed. Reason: %s" def __init__(self, uri, reason): - msg = self.__class__.msg_fmt % (uri, reason) + msg = self.msg_fmt % (uri, reason) super(BadStoreUri, self).__init__(msg) @@ -100,9 +100,7 @@ def wrap_exception(f): return f(*args, **kw) except Exception as e: if not isinstance(e, Error): - #exc_type, exc_value, exc_traceback = sys.exc_info() logging.exception(_('Uncaught exception')) - #logging.error(traceback.extract_stack(exc_traceback)) raise Error(str(e)) raise _wrap.func_name = f.func_name |
