mirror of
https://github.com/gryf/pythonhelper.git
synced 2025-12-19 12:28:16 +01:00
Added a FIXME for the 'except Exception'.
This commit is contained in:
@@ -628,6 +628,13 @@ def findTag(bufferNumber, changedTick):
|
|||||||
|
|
||||||
# handle possible exceptions {{{
|
# handle possible exceptions {{{
|
||||||
except Exception:
|
except Exception:
|
||||||
|
# FIXME: wrap try/except blocks around single sources of exceptions
|
||||||
|
# ONLY. Break this try/except block into as many small ones as you
|
||||||
|
# need, and only catch classes of exceptions that you have encountered.
|
||||||
|
# Catching "Exception" is very, very bad style!
|
||||||
|
# To the author: why is this clause here? There's no git log for why you
|
||||||
|
# have added it. Can you please put in a comment of a specific situation
|
||||||
|
# where you have encountered exceptions?
|
||||||
# bury into the traceback {{{
|
# bury into the traceback {{{
|
||||||
ec, ei, tb = sys.exc_info()
|
ec, ei, tb = sys.exc_info()
|
||||||
while (tb != None):
|
while (tb != None):
|
||||||
|
|||||||
Reference in New Issue
Block a user