On 21.03.2016 10:33, Christian Heimes wrote:
On 2016-03-21 10:29, Petr Spacek wrote:
On 20.3.2016 21:56, Martin Basti wrote:
I do not really like
because it replaces most of
try: ... except:
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.
It adds some value. A bare except also excepts signals like
KeyboardInterrupt and SystemExit. except Exception doesn't block these
But yes, more specific exceptions are better.
'except Exception' is another pylint check :D
I replaced bare except with a particular exception in cases where it was
clear. For other occurrences of bare except it covers too much Exception
types, so catch Exception is more sensible, or I need crystal ball to
detect what kind of exceptions can be raised there.
Manage your subscription for the Freeipa-devel mailing list:
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code