[Patches] [ python-Patches-410161 ] Create parsermodule.h
noreply@sourceforge.net
noreply@sourceforge.net
Fri, 23 Mar 2001 18:26:39 -0800
Patches item #410161, was updated on 2001-03-20 13:38
You can respond by visiting:
http://sourceforge.net/tracker/?func=detail&atid=305470&aid=410161&group_id=5470
Category: Modules
Group: None
Status: Open
Priority: 5
Submitted By: Paul Prescod (prescod)
Assigned to: Fred L. Drake, Jr. (fdrake)
Summary: Create parsermodule.h
Initial Comment:
(supercedes #409307)
When Fred created parsermodule.c back in the mid-90's
he put in a comment that some of the code should
really be moved into a header file for more general
use. Now I need that feature to implement compiler
hooks.
The basic idea is that parsermodule declared all of
its types and functions in the .c file instead of
a .h. That meant that they were unavailable to other
code.
----------------------------------------------------------------------
>Comment By: Paul Prescod (prescod)
Date: 2001-03-23 18:26
Message:
Logged In: YES
user_id=31788
Well here's some incentive to check it in when you get a
chance. I've got another patch that allows for pluggable
compilers. parsermodule is used to build node objects to
pass to Python code that replaces the compiler. Of course
we may soon want pluggable parsers too...
----------------------------------------------------------------------
Comment By: Fred L. Drake, Jr. (fdrake)
Date: 2001-03-20 22:17
Message:
Logged In: YES
user_id=3066
I'll try to check in something similar tomorrow, when I've
had a little time to sit on this now that I've actually read
the comments. ;-) I don't think the included header is
exactly right, but that's in part that I don't think the
names are good (yes, I know they're mine!).
----------------------------------------------------------------------
Comment By: Paul Prescod (prescod)
Date: 2001-03-20 13:41
Message:
Logged In: YES
user_id=31788
(argh...the patch didn't show up)
----------------------------------------------------------------------
Comment By: Paul Prescod (prescod)
Date: 2001-03-20 13:39
Message:
Logged In: YES
user_id=31788
I'm adding the changes to parsermodule.c that need to be
made so as not to duplicate the information in the new
parsermodule.h.
----------------------------------------------------------------------
You can respond by visiting:
http://sourceforge.net/tracker/?func=detail&atid=305470&aid=410161&group_id=5470