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

Tyson Dowd trd at cs.mu.OZ.AU
Tue Jul 17 19:10:30 AEST 2001


On 17-Jul-2001, Fergus Henderson <fjh at cs.mu.OZ.AU> wrote:
> 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.

Best way to do this is file a feature request and assign it to me.
I'll get it done once the backlog of things I'm working on now is fixed.

> > 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++".

Actually accepts "C" "c" "MC++" "Managed C++" "managed C++" "ManagedC++"
"csharp" "C#" "CSharp" etc, etc.

Does your proposed design allow these variants?

-- 
       Tyson Dowd           # 
                            #  Surreal humour isn't everyone's cup of fur.
     trd at cs.mu.oz.au        # 
http://www.cs.mu.oz.au/~trd #
--------------------------------------------------------------------------
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