Gmail Calendar Documents Reader Web more »
Recently Visited Groups | Help | Sign in
Google Groups Home
cvs commit: parrot/tools/dev parrot_api.pl
There are currently too many topics in this group that display first. To make this topic appear first, remove this option from another topic.
There was an error processing your request. Please try again.
flag
  2 messages - Collapse all  -  Translate all to Translated (View all originals)
The group you are posting to is a Usenet group. Messages posted to this group will make your email address visible to anyone on the Internet.
Your reply message has not been sent.
Your post was successful
 
From:
To:
Cc:
Followup To:
Add Cc | Add Followup-to | Edit Subject
Subject:
Validation:
For verification purposes please type the characters you see in the picture below or the numbers you hear by clicking the accessibility icon. Listen and type the numbers you hear
 
Leopold Toetsch  
View profile  
 More options Nov 9 2004, 5:34 pm
Newsgroups: perl.perl6.internals
From: l...@toetsch.at (Leopold Toetsch)
Date: Tue, 9 Nov 2004 23:34:03 +0100
Local: Tues, Nov 9 2004 5:34 pm
Subject: Re: cvs commit: parrot/tools/dev parrot_api.pl

Jarkko Hietaniemi <j...@cvs.perl.org> wrote:
>   +            if (/^\w+\s+(Parrot_\w+)\(/) {

Can we be slightly less strict? Current publics that ought to be APIs
include these prefixes:

IMCC_   PASM/PIR compiler stuff
AST_    AST      compiler stuff
PF_     Packfile handling low level
PackFile_  same/higher level, but needs review
PDB_    Parrot debugger
PIO_    Parrot IO

Another possible issue the program shows is: there are tons of public
symbols that have a Parrot_ preifx, which are *nether* API calls:

- Parrot opcode functions (core_ops.o)

and some may be embedding APIs:

- Parrot vtable functions

Thamks Jarkko,
leo


    Forward  
You must Sign in before you can post messages.
To post a message you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
Jarkko Hietaniemi  
View profile  
 More options Nov 10 2004, 1:45 am
Newsgroups: perl.perl6.internals
From: j...@iki.fi (Jarkko Hietaniemi)
Date: Wed, 10 Nov 2004 08:45:46 +0200
Local: Wed, Nov 10 2004 1:45 am
Subject: Re: cvs commit: parrot/tools/dev parrot_api.pl

Leopold Toetsch wrote:
> Jarkko Hietaniemi <j...@cvs.perl.org> wrote:

>>  +         if (/^\w+\s+(Parrot_\w+)\(/) {

> Can we be slightly less strict? Current publics that ought to be APIs
> include these prefixes:

That's a policy decision.  I would make a different policy decision
(that is, *everything* parrot exports would begin with Parrot, e.g.
ParrotC for compiler, ParrotD for debugger), but obviously I don't
make any policy decisions regarding Parrot.

> IMCC_   PASM/PIR compiler stuff
> AST_    AST      compiler stuff
> PF_     Packfile handling low level
> PackFile_  same/higher level, but needs review
> PDB_    Parrot debugger
> PIO_    Parrot IO

> Another possible issue the program shows is: there are tons of public
> symbols that have a Parrot_ preifx, which are *neither* API calls:

> - Parrot opcode functions (core_ops.o)

> and some may be embedding APIs:

> - Parrot vtable functions

The question is which of these "tons" do you want exposed?  The sad
truth is as soon as a symbol is exposed, someone will use it, and
then you are stuck with it, making it harder to change the interface
ever again.  Therefore minimizing the number of exposed symbols is
a worthy future-proofing task.  Also, I do not see *any* excuse
for exposing any symbol that doesn't have *any* of the approved
prefixes.

> Thanks Jarkko,
> leo

--
Jarkko Hietaniemi <j...@iki.fi> http://www.iki.fi/jhi/ "There is this special
biologist word we use for 'stable'.  It is 'dead'." -- Jack Cohen

    Forward  
You must Sign in before you can post messages.
To post a message you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
End of messages
« Back to Discussions « Newer topic     Older topic »

Create a group - Google Groups - Google Home - Terms of Service - Privacy Policy
©2010 Google