[m-rev.] for review: use new stream library in term_to_xml

Julien Fischer juliensf at csse.unimelb.edu.au
Mon Oct 30 18:19:31 AEDT 2006


On Mon, 30 Oct 2006, Ian MacLarty wrote:

> On Mon, Oct 30, 2006 at 04:40:00PM +1100, Julien Fischer wrote:
>>
>> On Mon, 30 Oct 2006, Ian MacLarty wrote:
>>
>>> Estimated hours taken: 5
>>> Branches: main
>>>
>>> Use the new stream typeclass in the term_to_xml standard library module.
>>> In the process remove some clutter from the interface by obsoleting
>>> the predicates that do not take a stream argument and remove the
>>> "_to_stream" suffix from those predicates that do take a stream
>>> argument.
>>>
>>> library/term_to_xml.m:
>>> 	Use the stream.writer/3 typeclass where appropriate.
>>>
>>> 	Make all XML writer predicates require a stream argument
>>> 	and remove the "_to_stream" suffix from these predicates.
>>>
>>> 	Move all deprecated predicates to the end of the interface and
>>> 	pragma obsolete them.
>>>
>>> 	Reword some comments.
>>>
>>> 	Remove the behaviour of replacing "]]>" with "]]>" in CDATA
>>> 	elements, since that behaviour is a bit misleading, because ">"
>>> 	has no special meaning in CDATA.  Instead just document that "]]>"
>>> 	is not allowed in CDATA elements.
>>>
>>> browser/browse.m:
>>> tests/hard_coded/write_xml.m:
>>> tests/hard_coded/xmlable_test.m:
>>> 	Conform to the above changes.
>>
>> The NEWS file will also need updating at some point, otherwise that's
>> fine.
>>
>
> Aren't we going put a general streams announcement in the NEWS file once
> all the streams stuff has been committed?

Yep, hence the "at some point" bit.

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



More information about the reviews mailing list