[m-rev.] for review: il as a foreign language

Fergus Henderson fjh at cs.mu.OZ.AU
Tue Jul 17 02:51:58 AEST 2001


On 17-Jul-2001, Tyson Dowd <trd at cs.mu.OZ.AU> wrote:
> On 17-Jul-2001, Fergus Henderson <fjh at cs.mu.OZ.AU> wrote:
> > > + at example
> > > +:- pragma foreign_proc("@var{Lang}", @var{Pred}(@var{Var1}::@var{Mode1}, @var{Var2}::@var{Mode2}, @dots{}),
> > > +        @var{Attributes}, @var{Foreign_Code}).
> > > + at end example
> > 
> > I think the syntax there should be changed from "Lang" to 'Lang'.
> > The set of supported languages is an enumeration, not an arbitrary
> > string literal, so we should use enumeration-like syntax rather than
> > string literal syntax.
> 
> I'd prefer to document what is implemented.
> If you want to change the syntax we can have a different discussion
> about that. 

OK, I don't mind if you documented what is implemented for now,
but I do think it should be changed.

> AFAI remember you suggested using strings in the first place

I'm pretty sure you remember wrongly on that point.

> (I was going to use 'c' and 'mcplusplus' to begin with),

And then I suggested 'C' and 'Managed C++'
and then for reasons that I don't really understand or agree with
you changed it to use string literals, lowercase "c", and "MC++".

-- 
Fergus Henderson <fjh at cs.mu.oz.au>  |  "I have always known that the pursuit
The University of Melbourne         |  of excellence is a lethal habit"
WWW: <http://www.cs.mu.oz.au/~fjh>  |     -- the last words of T. S. Garp.
--------------------------------------------------------------------------
mercury-reviews mailing list
post:  mercury-reviews at cs.mu.oz.au
administrative address: owner-mercury-reviews at cs.mu.oz.au
unsubscribe: Address: mercury-reviews-request at cs.mu.oz.au Message: unsubscribe
subscribe:   Address: mercury-reviews-request at cs.mu.oz.au Message: subscribe
--------------------------------------------------------------------------



More information about the reviews mailing list