Modify

Opened 7 years ago

#948 new defect

Do not use bare "except:"

Reported by: Erik Schnetter Owned by: Erik Schnetter
Priority: minor Milestone:
Component: SimFactory Version:
Keywords: Cc:

Description

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.

Attachments (0)

Change History (0)

Modify Ticket

Change Properties
Set your email in Preferences
Action
as new The owner will remain Erik Schnetter.
Next status will be 'review'.
as The resolution will be set.
to The owner will be changed from Erik Schnetter to the specified user.
Next status will be 'confirmed'.
The owner will be changed from Erik Schnetter to anonymous.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.