[Freeipa-devel] [PATCHES 0442-0449] Pylint: sunday code cleanup
Petr Spacek
pspacek at redhat.com
Mon Mar 21 09:29:29 UTC 2016
On 20.3.2016 21:56, Martin Basti wrote:
> Patches attached.
I do not really like
freeipa-mbasti-0442-pylint-remove-bare-except
because it replaces most of
try: ... except:
with
try: ... except Exception:
which AFAIK does not add any value. It would be better to replace Exception
with more specific exception so the code raises an error instead of continuing
when something really unexpected happens.
Other patches look sensible to me.
--
Petr^2 Spacek
More information about the Freeipa-devel
mailing list