[Patches] [Patch #103589] Fix segfault related to nested scopes
noreply@sourceforge.net
noreply@sourceforge.net
Fri, 09 Feb 2001 15:05:11 -0800
Patch #103589 has been updated.
Project: python
Category: Parser/Compiler
Status: Closed
Submitted by: nickm
Assigned to : jhylton
Summary: Fix segfault related to nested scopes
Follow-Ups:
Date: 2001-Feb-09 15:05
By: jhylton
Comment:
done
-------------------------------------------------------
Date: 2001-Feb-09 07:00
By: gvanrossum
Comment:
Jeremy, please note that the addition to test_scope.py mixes tabs and
spaces, which will make the test suite fail. Easy to fix of course...
-------------------------------------------------------
Date: 2001-Feb-08 22:08
By: gvanrossum
Comment:
Thanks, that's much better. It seems to work for me, but I'll leave it
open for Jeremy so he can proofread it!
-------------------------------------------------------
Date: 2001-Feb-08 21:52
By: nickm
Comment:
My apologies; Guido is correct about the example and the patch.
This patch should work against the latest CVS version. Again, it includes
test cases for the cases it fixes.
-------------------------------------------------------
Date: 2001-Feb-08 18:21
By: gvanrossum
Comment:
I agree with the poster, this is not fixed in CVS. Note that the example is
wrong: it only fails if you actually *call* the inner function.
To the poster: your diffs are malformed. Did you use cut-and-paste to
upload the diff? It seems some lines are missing. Please save the diff to
a file and use file upload (don't forget to click the "Upload Revised
Patch" checkbox!
-------------------------------------------------------
Date: 2001-Feb-06 06:14
By: nickm
Comment:
Just to clarify, if this problem is fixed, the fix isn't in CVS. I can
still
induce a segfault with:
def outer(*args):
def inner():
print args
outer(1,2,3)
This patch fixes this problem.
-------------------------------------------------------
Date: 2001-Feb-06 04:34
By: moshez
Comment:
assigning to Jeremy -- I think he fixed the problems
Jeremy, I think this patch is no longer relevant.
-------------------------------------------------------
Date: 2001-Feb-04 09:38
By: nickm
Comment:
Oops! That wasn't a context diff. This should be.
-------------------------------------------------------
Date: 2001-Feb-04 07:22
By: nickm
Comment:
Improved patch; original patch modified unreleated code.
-------------------------------------------------------
-------------------------------------------------------
For more info, visit:
http://sourceforge.net/patch/?func=detailpatch&patch_id=103589&group_id=5470