[m-rev.] For review: Convert Mercury terms to XML

Zoltan Somogyi zs at cs.mu.OZ.AU
Mon Dec 6 11:55:03 AEDT 2004


On 06-Dec-2004, Ralph Becket <rafe at cs.mu.OZ.AU> wrote:
> > I'm not really keen on the name `to_xml'.  How about either
> > `xml' or `xml_util'?
> 
> I'm unhappy with all three suggestions.
> term_to_xml would describe what it does.

I am happy with either to_xml or term_to_xml.

> > > Index: library/to_xml.m
> 
> I don't think this is of sufficiently general utility to belong in the
> standard library.  It should go in extras.

I think it is OK to go into the standard library.

> > Using memoing in the standard library is going to mean that it no
> > longer compiles for those grades that don't support it, e.g. the java
> > and il grades.
> 
> I agree.  It's not a good idea.  For a start, memo tables persist
> through GC, so they are likely to be a type of space leak for
> long-running applications (this is a hole in the memoing API IMHO).
> 
> I'd leave it to the user to add memoed wrapper predicates if they need
> them.

I agree with Ralph here on general principles, but I would like to know
what the efficiency hit for the C backend is if you take out the pragma
memo.

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