Replace instead of add a xml node

Question

In this simple code after invoking second(), 1.xml has only one node "1". Why pugi replaces node and what should I do for correct modifying?

void first()
{
    pugi::xml_document document;
    pugi::xml_parse_result result = document.load_file("C:\\1.xml", parse_full);
    pugi::xml_node node = document.append_child("0");
    node.append_attribute("message") = "something";
    document.save_file("C:\\1.xml");
}
void second()
{
    pugi::xml_document document;
    pugi::xml_parse_result result = document.load_file("C:\\1.xml", parse_full);
    pugi::xml_node node = document.append_child("1");
    node.append_attribute("message") = "something else";
    document.save_file("C:\\1.xml");
}
void test()
{
    first();
    second();
}

i would check the documentation of the library.

Cheers and hth. - Alf
– Cheers and hth. - Alf

2014-05-11 07:20:45 +00:00
Commented May 11, 2014 at 7:20 — Cheers and hth. - Alf
– Cheers and hth. - Alf, Commented May 11, 2014 at 7:20

zeuxcg · Accepted Answer · 2014-05-11 16:18:31Z

1

You should check the result of load_file.

Here's what's going on here:

XML tag names can't start with digits. This is defined by XML standard.
pugixml performs this check while loading the document - so load_file() fails, producing an empty document
pugixml does not perform this check while appending nodes or saving document, so it is possible to save an invalid document

answered May 11, 2014 at 16:18

zeuxcg

9,4741 gold badge31 silver badges34 bronze badges

Sign up to request clarification or add additional context in comments.

Collectives™ on Stack Overflow

Replace instead of add a xml node

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related