Re: [lmi] change file formats to XML

[Greg Chicares]

On 2010-03-09 17:53Z, Vaclav Slavik wrote:
> On Mon, 2010-03-08 at 16:25 +0100, Vaclav Slavik wrote:
>> I'd argue it is: I'd expect xml::node(xml::node::text("M&E")) to write
>> "M&E" into the file. Or maybe throw, but certainly not write "M"
>> silently and pretend everything is fine. I have similar expectation from
>> xml::node::set_content(), but that one may be less clear.
> 
> Turns out that this behavior of set_content() is intentional. I'll add a
> replacement in xmlwrapp 0.7; for now, please replace the two occurrences
> of set_content() in my patch with direct inclusion of text node:
[...]

Okay, thanks: it works correctly; it appears to have no material
effect on runtime speed; and it obviates the changes (to non-
published code) that I had mentioned earlier:

On 2010-03-06 00:16Z, Greg Chicares wrote:
|
| Explanation: for now, we still have some non-published code that uses
| the published product-file code; it writes files in the new format,
| but with the old extensions. [Some of those files contain literal
| ampersands, which must be replaced with xml '&' entities; after
| replacement in the non-published source, we see:
|   Income & Growth
|   M&E
| rather than:
|   Income & Growth
|   M&E
| .]