Do not use bare "except:"

Create issue
Issue #948 resolved
Erik Schnetter created an issue

A bare "except:" (without listing an exception class) will intercept all exceptions, including the user typing control-C to abort. This should therefore not be used, except if (a) the code will abort anyway with an appropriate user-visible error message, or (b) the code will re-raise the exception via "raise".

In particular, Simfactory catches filesystem errors with bare excepts; this should be cleaned up.


Comments (5)

  1. Log in to comment