IntenseWebs/freeipa
3
0
Fork 0
mirror of https://salsa.debian.org/freeipa-team/freeipa.git synced 2025-02-25 18:55:28 -06:00
Code Issues Packages Projects Releases Wiki Activity

Enable pylint lost exception check

Browse Source 
Commit enables check and also fixes:
ipaserver/dcerpc.py:718: [W0150(lost-exception),
DomainValidator.__search_in_dc] return statement in finally block may
swallow exception)

Reviewed-By: Jan Cholasta <jcholast@redhat.com>
This commit is contained in:
Martin Basti 2015-12-16 23:26:29 +01:00 committed by Jan Cholasta
parent 837e8f8160
commit fb3eab7ec7
2 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
ipaserver/dcerpc.py
View File

@ -714,8 +714,8 @@ class DomainValidator(object):
root_logger.debug(msg)
else:
root_logger.warning(msg)
finally:
return entries
return entries
def __retrieve_trusted_domain_gc_list(self, domain):
"""

1
make-lint
View File

@ -264,7 +264,6 @@ def main():
linter.set_option('disable', 'global-statement')
linter.set_option('disable', 'global-variable-not-assigned')
linter.set_option('disable', 'global-variable-undefined')
linter.set_option('disable', 'lost-exception')
linter.set_option('disable', 'no-init')
linter.set_option('disable', 'pointless-except')
linter.set_option('disable', 'pointless-statement')