[Python-bugs-list] [ python-Bugs-411881 ] Use of "except:" in modules
noreply@sourceforge.net
noreply@sourceforge.net
Fri, 22 Mar 2002 22:02:58 -0800
Bugs item #411881, was opened at 2001-03-28 06:58
You can respond by visiting:
http://sourceforge.net/tracker/?func=detail&atid=105470&aid=411881&group_id=5470
Category: Python Library
Group: None
Status: Open
Resolution: None
Priority: 3
Submitted By: Itamar Shtull-Trauring (itamar)
Assigned to: Skip Montanaro (montanaro)
>Summary: Use of "except:" in modules
Initial Comment:
A large amount of modules in the standard library use
"except:" instead of specifying the exceptions to be
caught. In some cases this may be correct, but I think
in most cases this not true and this may cause
problems. Here's the list of modules, which I got by
doing:
grep "except:" *.py | cut -f 1 -d " " | sort | uniq
Bastion.py
CGIHTTPServer.py
Cookie.py
SocketServer.py
anydbm.py
asyncore.py
bdb.py
cgi.py
chunk.py
cmd.py
code.py
compileall.py
doctest.py
fileinput.py
formatter.py
getpass.py
htmllib.py
imaplib.py
inspect.py
locale.py
locale.py
mailcap.py
mhlib.py
mimetools.py
mimify.py
os.py
pdb.py
popen2.py
posixfile.py
pre.py
pstats.py
pty.py
pyclbr.py
pydoc.py
repr.py
rexec.py
rfc822.py
shelve.py
shutil.py
tempfile.py
threading.py
traceback.py
types.py
unittest.py
urllib.py
zipfile.py
----------------------------------------------------------------------
>Comment By: Skip Montanaro (montanaro)
Date: 2002-03-23 00:02
Message:
Logged In: YES
user_id=44345
as partial fix, checked in changes for the following
modules:
mimetools.py (1.24)
popen2.py (1.23)
quopripy (1.19)
CGIHTTPServer.py (1.22)
----------------------------------------------------------------------
Comment By: Skip Montanaro (montanaro)
Date: 2002-03-20 15:24
Message:
Logged In: YES
user_id=44345
Here is a context diff with proposed changes for the
following modules: CGIHTTPServer, cgi, cmd, code,
fileinput, httplib, inspect, locale, mimetools, popen2,
quopri
----------------------------------------------------------------------
Comment By: Martin v. Löwis (loewis)
Date: 2001-08-11 10:06
Message:
Logged In: YES
user_id=21627
Fixed urllib in 1.131 and types in 1.19.
----------------------------------------------------------------------
Comment By: Fred L. Drake, Jr. (fdrake)
Date: 2001-07-04 02:11
Message:
Logged In: YES
user_id=3066
Fixed modules mhlib and rfc822 (SF is having a problem
generating the checkin emails, though).
----------------------------------------------------------------------
Comment By: Fred L. Drake, Jr. (fdrake)
Date: 2001-05-11 14:40
Message:
Logged In: YES
user_id=3066
OK, I've fixed up a few more modules:
anydbm
chunk
formatter
htmllib
mailcap
pre
pty
I made one change to asyncore as well, but other bare except
clauses remain there; I'm not sufficiently familiar with
that code to just go digging into those.
I also fixed an infraction in pstats, but left others for now.
----------------------------------------------------------------------
Comment By: Tim Peters (tim_one)
Date: 2001-04-23 03:14
Message:
Logged In: YES
user_id=31435
Ping's intent is that pydoc work under versions of Python
as early as 1.5.2, so that sys._getframe is off-limits in
pydoc and its supporting code (like inspect.py).
----------------------------------------------------------------------
Comment By: Martin v. Löwis (loewis)
Date: 2001-04-23 02:32
Message:
Logged In: YES
user_id=21627
For inspect.py, why is it necessary to keep the old code at
all? My proposal: remove currentframe altogether, and do
currentframe = sys._getframe
unconditionally.
----------------------------------------------------------------------
Comment By: Itamar Shtull-Trauring (itamar)
Date: 2001-04-22 09:52
Message:
Logged In: YES
user_id=32065
I submitted a 4th patch. I'm starting to run out of easy
cases...
----------------------------------------------------------------------
Comment By: Skip Montanaro (montanaro)
Date: 2001-04-19 04:15
Message:
Logged In: YES
user_id=44345
I believe the following patch is correct for the try/except
in inspect.currentframe. Note that it fixes two problems.
One, it avoids a bare except. Two, it gets rid of a string
argument to the raise statement (string exceptions are now
deprecated, right?).
*** /tmp/skip/inspect.py Thu Apr 19 04:13:36 2001
--- /tmp/skip/inspect.py.~1.16~ Thu Apr 19 04:13:36 2001
***************
*** 643,650 ****
def currentframe():
"""Return the frame object for the caller's stack
frame."""
try:
! 1/0
! except ZeroDivisionError:
return sys.exc_traceback.tb_frame.f_back
if hasattr(sys, '_getframe'): currentframe = sys._getframe
--- 643,650 ----
def currentframe():
"""Return the frame object for the caller's stack
frame."""
try:
! raise 'catch me'
! except:
return sys.exc_traceback.tb_frame.f_back
if hasattr(sys, '_getframe'): currentframe = sys._getframe
----------------------------------------------------------------------
Comment By: Itamar Shtull-Trauring (itamar)
Date: 2001-04-17 10:27
Message:
Logged In: YES
user_id=32065
inspect.py uses sys_getframe if it's there, the other code
is for backwards compatibility.
----------------------------------------------------------------------
Comment By: Guido van Rossum (gvanrossum)
Date: 2001-04-11 12:24
Message:
Logged In: YES
user_id=6380
Actually, inspect.py should use sys._getframe()!
And yes, KeyboardError is definitely one of the reasons why
this is such a bad idiom...
----------------------------------------------------------------------
Comment By: Walter Dörwald (doerwalter)
Date: 2001-04-11 12:15
Message:
Logged In: YES
user_id=89016
> Can you identify modules where catching everything
> is incorrect
If "everything" includes KeyboardInterrupt, it's
definitely incorrect, even in inspect.py's simple
try:
raise 'catch me'
except:
return sys.exc_traceback.tb_frame.f_back
which should probably be:
try:
raise 'catch me'
except KeyboardInterrupt:
raise
except:
return sys.exc_traceback.tb_frame.f_back
----------------------------------------------------------------------
Comment By: Walter Dörwald (doerwalter)
Date: 2001-04-11 12:13
Message:
Logged In: YES
user_id=89016
> Can you identify modules where catching everything
> is incorrect
If "everything" includes KeyboardInterrupt, it's
definitely incorrect, even in inspect.py's simple
try:
raise 'catch me'
except:
return sys.exc_traceback.tb_frame.f_back
which should probably be:
try:
raise 'catch me'
except KeyboardInterrupt:
raise
except:
return sys.exc_traceback.tb_frame.f_back
----------------------------------------------------------------------
Comment By: Guido van Rossum (gvanrossum)
Date: 2001-04-10 10:45
Message:
Logged In: YES
user_id=6380
I've applied the three patches you supplied.
I agree with Martin that to do this right we'll have to
tread carefully. But please go on!
(No way more of this will find its way into 2.1 though.)
----------------------------------------------------------------------
Comment By: Itamar Shtull-Trauring (itamar)
Date: 2001-03-30 04:54
Message:
Logged In: YES
user_id=32065
inspect.py should be removed from this list, the use is
correct.
In general, I just submitted this bug so that when people
are editing a module they'll notice these things, since in
some cases only someone who knows the code very well can
know if the "expect:" is needed or not.
----------------------------------------------------------------------
Comment By: Martin v. Löwis (loewis)
Date: 2001-03-30 00:59
Message:
Logged In: YES
user_id=21627
Can you identify modules where catching everything is
incorrect, and propose changes to correct them. This should
be done one-by-one, with careful analysis in each case, and
may take well months or years to complete.
----------------------------------------------------------------------
You can respond by visiting:
http://sourceforge.net/tracker/?func=detail&atid=105470&aid=411881&group_id=5470