[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