[m-rev.] for review: shift spawn/3 etc, into the standard library

[Julien Fischer]

On Mon, 15 Jan 2007, Peter Wang wrote:

> On 2007-01-15, Julien Fischer <juliensf at csse.unimelb.edu.au> wrote:
>>
>> For review by Peter Wang.
>>
>> The name of the new library module hasn't been finalised yet but
>> I would like comments on the rest of it.
>>
>> Estimated hours taken: 2
>> Branches: main
>>
>> Shift extras/concurrency/spawn.m into a new standard library module
>> named `thread'.
>
> If later we move some of the other modules from extras/concurrency into
> the standard library, would they be in submodules, e.g. thread.mvar?

Now that we support sub-modules that seems like the obvious thing to do.
(Although I haven't given it too much thought.)


>> library/thread.m:
>> 	New module.  This contains modified versions of the spawn/3
>> 	and yield/2 predicates that previously lived in
>> 	extras/concurrency/spawn.m.  The main modifications are:
>>
>> 	- we now allow this module to compile in non .par hl* grades.
>> 	  (Attempting to acutally create a new thread in this grades will
>
> actually

Fixed.

>> NEWS:
>> 	Annouce the change.
>
> Announce

Fixed.

>> Index: NEWS
>> ===================================================================
>> RCS file: /home/mercury/mercury1/repository/mercury/NEWS,v
>> retrieving revision 1.438
>> diff -u -r1.438 NEWS
>> --- NEWS	12 Jan 2007 05:00:32 -0000	1.438
>> +++ NEWS	15 Jan 2007 11:53:11 -0000
>> @@ -17,6 +17,9 @@
>>
>>  Changes to the Mercury standard library:
>>
>> +* We have moved the concurrency primitives out of the extras distribution
>> and
>> +  into a new standard library module called `thread'.
>
> moved some of the concurrency

Done.

> The MCFLAGS setting should also be moved from
> extras/concurrency/Mercury.options.  The rest looked fine.

Done.  I won't commit this for a few days as I want to check that
the compiler still boostraps in a number of grades (specifically 
I want to make sure that the library builds properly.)

Julien.
--------------------------------------------------------------------------
mercury-reviews mailing list
Post messages to:       mercury-reviews at csse.unimelb.edu.au
Administrative Queries: owner-mercury-reviews at csse.unimelb.edu.au
Subscriptions:          mercury-reviews-request at csse.unimelb.edu.au
--------------------------------------------------------------------------