[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