[Patches] [ python-Patches-1176019 ] Implemented new 'class foo():pass' syntax

SourceForge.net noreply at sourceforge.net
Fri Apr 15 04:19:04 CEST 2005


Patches item #1176019, was opened at 2005-04-03 21:47
Message generated for change (Comment added) made by nascheme
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=1176019&group_id=5470

Category: Parser/Compiler
Group: AST
>Status: Closed
>Resolution: Accepted
Priority: 5
Submitted By: logistix (logistix)
>Assigned to: Neil Schemenauer (nascheme)
Summary: Implemented new 'class foo():pass' syntax

Initial Comment:
This patch implements the new 'class foo():pass' syntax 
for the ast branch.  It also includes minor modifications 
required to run build graminit.c on windows machines.

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

>Comment By: Neil Schemenauer (nascheme)
Date: 2005-04-15 02:19

Message:
Logged In: YES 
user_id=35752

Looks good.  Check into the AST branch.  Thanks for the patch.

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

Comment By: logistix (logistix)
Date: 2005-04-04 21:49

Message:
Logged In: YES 
user_id=699438

Just to confirm, pgen did generate a new graminit.h file, but 
there were no changes to it because we didn't add any new 
symbols.  The patch I submitted is fine.

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

Comment By: logistix (logistix)
Date: 2005-04-04 20:53

Message:
Logged In: YES 
user_id=699438

graminit.c is generated by pgen.exe.  This process isn't fully 
automated on a windows build (like I'm guessing it is on 
linux).  There is a provided makefile that works with visual 
studio's nmake for windows users.  It just referenced an older 
version of the python lib file (2.3 instead of 2.5).

I presume this hasn't caused problems in the past becuase 
people modifying grammar/grammar have been running some 
form of *nix.

Now I'm starting to wonder if pgen generates any other files 
(i.e. header files) that I didn't include in the patch.  I'll verify 
when I get home.

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

Comment By: Brett Cannon (bcannon)
Date: 2005-04-04 19:25

Message:
Logged In: YES 
user_id=357491

Do you know why graminit.c stopped working on Windows? 
Kurt's report on the merger didn't show any issues on
Windows and so I wonder if the problem goes back to the main
branch as well.

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

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


More information about the Patches mailing list