[m-rev.] [dotnet-foreign] diff: use System.Array for mercury arrays

Fergus Henderson fjh at cs.mu.OZ.AU
Thu Aug 2 23:34:12 AEST 2001


On 02-Aug-2001, Tyson Dowd <trd at cs.mu.OZ.AU> wrote:
> This is becoming a big enough task that I might leave it for a
> while (I have some other things to do for a bit).  Everything is pretty
> easy *except* allocating arrays.

It's only array__make_empty_array that is difficult.
array__init is easy.  The code currently there is almost enough,
you just need to add

	for (int i = 0; i < Size; i++) {
		Array->SetValue(Item, i);
	}

> I will post my changes to the compiler, and then leave off the library
> support for a little while.

I think it would be easier to do the library support now while the
concepts are all still in your head ;-)

I agree with leaving the RTTI support and hence array__make_empty_array
for a later change, but if you're going to touch arrays at all, then
I think it's worth at least fixing array__set, array__lookup,
and array__init.

-- 
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