diff options
author | Lukas Slebodnik <lslebodn@redhat.com> | 2015-02-09 16:46:05 +0100 |
---|---|---|
committer | Jakub Hrozek <jhrozek@redhat.com> | 2015-02-13 18:31:39 +0100 |
commit | 1ac368d0962ef8cc83dcd642c7fec8b3cba5b6fe (patch) | |
tree | 66565eb01b28ed6b9114b10e8273b3f864696fd2 /src | |
parent | b0f46a3019e0ff4f375ef07682ceb9418751707f (diff) | |
download | sssd-1ac368d0962ef8cc83dcd642c7fec8b3cba5b6fe.tar.gz sssd-1ac368d0962ef8cc83dcd642c7fec8b3cba5b6fe.tar.xz sssd-1ac368d0962ef8cc83dcd642c7fec8b3cba5b6fe.zip |
SSSDConfig: Remove unused exception name
"except ValueError, e:" was the syntax used for what is normally written
as "except ValueError as e:" in modern Python. The old syntax is still
supported in python2 for backwards compatibility.
This means "except ValueError, KeyError:" is not equivalent to
"except (ValueError, KeyError):" but to "except ValueError as KeyError:"
and variable with name "KeyError" was not used in exception handler.
Resolves:
https://fedorahosted.org/sssd/ticket/2017
Reviewed-by: Petr Viktorin <pviktori@redhat.com>
Diffstat (limited to 'src')
-rw-r--r-- | src/config/SSSDConfig/__init__.py.in | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/config/SSSDConfig/__init__.py.in b/src/config/SSSDConfig/__init__.py.in index ae00a2b7f..63d1f177b 100644 --- a/src/config/SSSDConfig/__init__.py.in +++ b/src/config/SSSDConfig/__init__.py.in @@ -527,7 +527,7 @@ class SSSDConfigSchema(SSSDChangeConf): mandatory, desc, [subtype(split_option[DEFAULT])]) - except ValueError, KeyError: + except ValueError: raise ParsingError else: try: @@ -546,7 +546,7 @@ class SSSDConfigSchema(SSSDChangeConf): mandatory, desc, primarytype(split_option[DEFAULT])) - except ValueError, KeyError: + except ValueError: raise ParsingError elif optionlen > 4: @@ -561,7 +561,7 @@ class SSSDConfigSchema(SSSDChangeConf): else: newvalue = subtype(x) fixed_options.extend([newvalue]) - except ValueError, KeyError: + except ValueError: raise ParsingError else: fixed_options.extend([x]) |