X-Git-Url: https://git.sesse.net/?a=blobdiff_plain;f=c%2B%2B11%2Fxml-template.cpp;h=4f0c9fef6e903aa4a9772de64e176d612dd3c291;hb=1725768e2eaf68a1445f1aef9d95da0bf461e88b;hp=2e8fb9429fa3ab4508f33dcd4cd106ac75c39f52;hpb=6a37870125901165d10cc934d139fa259ff98ef5;p=xml-template

diff --git a/c++11/xml-template.cpp b/c++11/xml-template.cpp
index 2e8fb94..4f0c9fe 100644
--- a/c++11/xml-template.cpp
+++ b/c++11/xml-template.cpp
@@ -14,12 +14,11 @@ void clean_node(xmlNode *node)
 	}
 	if (node->ns != NULL &&
 	    strcmp(reinterpret_cast<const char *>(node->ns->href), "http://template.sesse.net/") == 0) {
-		xmlNode *frag = xmlNewDocFragment(node->doc);
-		xmlReplaceNode(node, frag);
-		frag->children = node->children;
-		frag->last = node->last;
+		while (node->children != NULL) {
+			xmlAddPrevSibling(node, node->children);
+		}
 
-		node->children = node->last = NULL;
+		xmlUnlinkNode(node);
 		xmlFreeNode(node);
 	}
 }
@@ -190,9 +189,8 @@ void Substitute::process(xmlNode *node, bool clean)
 			}
 
 			// Check all substitutions to see if we found anything appropriate.
+			string tag = reinterpret_cast<const char *>(child->name);
 			for (auto it : substitution_map) {
-				string tag = reinterpret_cast<const char *>(child->name);
-
 				// Attribute substitution.
 				if (begins_with(it.first, tag + "/")) {
 					const xmlChar *attr_key = reinterpret_cast<const xmlChar *>(
@@ -202,7 +200,7 @@ void Substitute::process(xmlNode *node, bool clean)
 					xmlSetProp(child, attr_key, attr_value);
 				} else if ((!id.empty() && begins_with(it.first, "#" + id + "/"))) {
 					const xmlChar *attr_key = reinterpret_cast<const xmlChar *>(
-						it.first.c_str() + tag.size() + 2);
+						it.first.c_str() + id.size() + 2);
 					const xmlChar *attr_value = reinterpret_cast<const xmlChar *>(
 						it.second->get_contents().c_str());
 					xmlSetProp(child, attr_key, attr_value);