[Patches] [ python-Patches-1100563 ] ast-branch: fix for coredump from new import grammar

SourceForge.net noreply at sourceforge.net
Mon Jan 17 15:16:57 CET 2005


Patches item #1100563, was opened at 2005-01-11 19:50
Message generated for change (Comment added) made by nnorwitz
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=1100563&group_id=5470

Category: Parser/Compiler
Group: AST
Status: Closed
Resolution: Out of Date
Priority: 5
Submitted By: logistix (logistix)
Assigned to: Kurt B. Kaiser (kbk)
Summary: ast-branch: fix for coredump from new import grammar

Initial Comment:
New import grammar causes a coredump in the ast-
branch.  Attached is a fix.

After this is applied we start getting coredumps from 
generator expressions in the test suite.  I'm reasonably 
sure (and correct me if I'm wrong) that the generator 
expressions fix is more elaborate, as it requires patches 
to the bytecode generation as well.

----------------------------------------------------------------------

>Comment By: Neal Norwitz (nnorwitz)
Date: 2005-01-17 09:16

Message:
Logged In: YES 
user_id=33168

Sorry, I didn't realize this patch existed before fixing
import.  This patch looks like it might be a bit cleaner.

As for the genexpr fix, yes, it is much more involved.  I
added some support to move things along.  At this point
what's required is to use the original patches and mold into
the new AST code.  The original checkin that is most
important is here:

http://mail.python.org/pipermail/python-checkins/2004-May/040900.html

----------------------------------------------------------------------

Comment By: Kurt B. Kaiser (kbk)
Date: 2005-01-16 23:05

Message:
Logged In: YES 
user_id=149084

Neal has checked in a fix to ast.c which corrects the
coredump due to import errors.  This dump was first
seen after merging trunk to ast-branch 07Jan05.  This
patch is out of date and at this point represents a 
different refactoring than what Neal has done.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=1100563&group_id=5470


More information about the Patches mailing list