[m-rev.] for review: move dl.m closure creation into runtime

Simon Taylor stayl at cs.mu.OZ.AU
Wed Sep 24 10:57:47 AEST 2003


On 22-Sep-2003, Fergus Henderson <fjh at cs.mu.OZ.AU> wrote:
> On 20-Sep-2003, Simon Taylor <stayl at cs.mu.OZ.AU> wrote:
> > 
> > browser/dl.m:
> > runtime/mercury_ho_call.{c,h}:
> > 	Move the code in dl.m to create closures into the runtime.
> > 	The code is actually clarified by implementing it entirely
> > 	in C, rather than a mish-mash of C and Mercury.
> > 
> > 	The rationale for this change is that Aditi needs to be able
> > 	to create closures for dynamically linked procedures.  Just
> > 	copying browser/dl.m as it was into the Aditi sources would
> > 	create an undesirable dependency on Mercury runtime
> > 	implementation details.
> 
> Shouldn't Aditi use extras/dynamic_linking/dl.m?

Possibly, altough it's simpler to use MR_create_closure() than
requiring Aditi users to install an extra library.

Even not considering Aditi, I think this change is an improvement
on the status quo, so I'll commit it.

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