[Patches] [Patch #103589] Fix segfault related to nested scopes

noreply@sourceforge.net noreply@sourceforge.net
Thu, 08 Feb 2001 21:52:31 -0800


Patch #103589 has been updated. 

Project: python
Category: Parser/Compiler
Status: Accepted
Submitted by: nickm
Assigned to : jhylton
Summary: Fix segfault related to nested scopes

Follow-Ups:

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