[m-rev.] for review: get Mercury working on Google App Engine

Peter Wang novalazy at gmail.com
Mon Sep 21 15:07:54 AEST 2009


On 2009-09-21, Ian MacLarty <maclarty at csse.unimelb.edu.au> wrote:
> For review by anyone.
> 
> Estimated hours taken: 1
> Branches: main
> 
> Get the standard library working on Google App Engine.
> 
> library/io.m:
>     Do not use AtomicInteger for ML_next_stream_id.
>     Instead use a synchronized method to increment ML_next_stream_id.
>     AtomicInteger is not supported on App Engine.
> 
>     Do not statically initialize mercury_stdin_binary and
>     mercury_stdout_binary.  The initialization code doesn't work on App
>     Engine.  Instead initialize these fields when they are required.

That looks fine.

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



More information about the reviews mailing list