--- parser3/src/classes/xnode.C	2001/10/16 14:43:30	1.7
+++ parser3/src/classes/xnode.C	2001/10/31 14:01:44	1.16
@@ -4,7 +4,7 @@
 	Copyright (c) 2001 ArtLebedev Group (http://www.artlebedev.com)
 	Author: Alexander Petrosyan <paf@design.ru> (http://design.ru/paf)
 
-	$Id: xnode.C,v 1.7 2001/10/16 14:43:30 parser Exp $
+	$Id: xnode.C,v 1.16 2001/10/31 14:01:44 paf Exp $
 */
 #include "classes.h"
 #ifdef XML
@@ -32,7 +32,7 @@ XalanNode& as_node(Pool& pool, const Str
 						int index, const char *msg) {
 	Value& value=params->as_no_junction(index, msg);
 	if(strcmp(value.type(), VXNODE_TYPE)!=0)
-		PTHROW(0, 0,
+		throw Exception(0, 0,
 			&method_name,
 			msg);
 
@@ -47,7 +47,7 @@ XalanAttr& as_attr(Pool& pool, const Str
 	XalanNode& node=as_node(pool, method_name, params, index, msg);
 
 	if(node.getNodeType()!=XalanNode::ATTRIBUTE_NODE)
-		PTHROW(0, 0, 
+		throw Exception(0, 0, 
 			&method_name,
 			msg);
 
@@ -62,15 +62,19 @@ XalanAttr& as_attr(Pool& pool, const Str
 static void _insertBefore(Request& r, const String& method_name, MethodParams *params) {
 	Pool& pool=r.pool();
 	VXnode& vnode=*static_cast<VXnode *>(r.self);
-	XalanNode& node=vnode.get_node(pool, &method_name);
+	XalanNode& selfNode=vnode.get_node(pool, &method_name);
 
 	XalanNode& newChild=as_node(pool, method_name, params, 0, "newChild must be node");
 	XalanNode& refChild=as_node(pool, method_name, params, 1, "refChild must be node");
 	
 	try {
-		node.insertBefore(&newChild, &refChild);
+		if(XalanNode *retNode=selfNode.insertBefore(&newChild, &refChild)) {
+			// write out result
+			VXnode& result=*new(pool) VXnode(pool, retNode, false);
+			r.write_no_lang(result);		
+		}
 	} catch(const XalanDOMException& e)	{
-		pool.exception()._throw(pool, &method_name, e);
+		Exception::provide_source(pool, &method_name, e);
 	}
 }
 
@@ -78,15 +82,19 @@ static void _insertBefore(Request& r, co
 static void _replaceChild(Request& r, const String& method_name, MethodParams *params) {
 	Pool& pool=r.pool();
 	VXnode& vnode=*static_cast<VXnode *>(r.self);
-	XalanNode& node=vnode.get_node(pool, &method_name);
+	XalanNode& selfNode=vnode.get_node(pool, &method_name);
 
 	XalanNode& newChild=as_node(pool, method_name, params, 0, "newChild must be node");
 	XalanNode& refChild=as_node(pool, method_name, params, 1, "refChild must be node");
 	
 	try {
-		node.replaceChild(&newChild, &refChild);
+		if(XalanNode *retNode=selfNode.replaceChild(&newChild, &refChild)) {
+			// write out result
+			VXnode& result=*new(pool) VXnode(pool, retNode, false);
+			r.write_no_lang(result);		
+		}
 	} catch(const XalanDOMException& e)	{
-		pool.exception()._throw(pool, &method_name, e);
+		Exception::provide_source(pool, &method_name, e);
 	}
 }
 
@@ -94,14 +102,18 @@ static void _replaceChild(Request& r, co
 static void _removeChild(Request& r, const String& method_name, MethodParams *params) {
 	Pool& pool=r.pool();
 	VXnode& vnode=*static_cast<VXnode *>(r.self);
-	XalanNode& node=vnode.get_node(pool, &method_name);
+	XalanNode& selfNode=vnode.get_node(pool, &method_name);
 
 	XalanNode& oldChild=as_node(pool, method_name, params, 0, "oldChild must be node");
 	
 	try {
-		node.removeChild(&oldChild);
+		if(XalanNode *retNode=selfNode.removeChild(&oldChild)) {
+			// write out result
+			VXnode& result=*new(pool) VXnode(pool, retNode, false);
+			r.write_no_lang(result);		
+		}
 	} catch(const XalanDOMException& e)	{
-		pool.exception()._throw(pool, &method_name, e);
+		Exception::provide_source(pool, &method_name, e);
 	}
 }
 
@@ -109,14 +121,18 @@ static void _removeChild(Request& r, con
 static void _appendChild(Request& r, const String& method_name, MethodParams *params) {
 	Pool& pool=r.pool();
 	VXnode& vnode=*static_cast<VXnode *>(r.self);
-	XalanNode& node=vnode.get_node(pool, &method_name);
+	XalanNode& selfNode=vnode.get_node(pool, &method_name);
 
 	XalanNode& newChild=as_node(pool, method_name, params, 0, "newChild must be node");
 	
 	try {
-		node.appendChild(&newChild);
+		if(XalanNode *retNode=selfNode.appendChild(&newChild)) {
+			// write out result
+			VXnode& result=*new(pool) VXnode(pool, retNode, false);
+			r.write_no_lang(result);		
+		}
 	} catch(const XalanDOMException& e)	{
-		pool.exception()._throw(pool, &method_name, e);
+		Exception::provide_source(pool, &method_name, e);
 	}
 }
 
@@ -133,7 +149,7 @@ static void _hasChildNodes(Request& r, c
 }
 
 // Node cloneNode(in boolean deep);
-/// @test 1. who frees it up? 2. ownerDocument=?
+/// @test ownerDocument=?
 static void _cloneNode(Request& r, const String& method_name, MethodParams *params) {
 	Pool& pool=r.pool();
 	VXnode& vnode=*static_cast<VXnode *>(r.self);
@@ -142,7 +158,7 @@ static void _cloneNode(Request& r, const
 	bool deep=params->as_bool(0, "deep must be bool", r);
 
 	// write out result
-	VXnode& result=*new(pool) VXnode(pool, node.cloneNode(deep));
+	VXnode& result=*new(pool) VXnode(pool, node.cloneNode(deep), true/*all sense goes here*/);
 	result.set_name(method_name);
 	r.write_no_lang(result);
 }
@@ -155,7 +171,7 @@ XalanElement& get_self_element(Request&
 	XalanNode& node=vnode.get_node(pool, &method_name);
 
 	if(node.getNodeType()!=XalanNode::ELEMENT_NODE)
-		PTHROW(0, 0, 
+		throw Exception(0, 0, 
 			&method_name,
 			"method can be called on node of ELEMENT type");
 
@@ -166,9 +182,9 @@ XalanElement& get_self_element(Request&
 static void _getAttribute(Request& r, const String& method_name, MethodParams *params) {
 	Pool& pool=r.pool();
 	XalanElement& element=get_self_element(r, method_name);
-	const char *name=params->as_string(0, "name must not be code").cstr(String::UL_AS_IS);
+	const String& name=params->as_string(0, "name must be string");
 
-	const XalanDOMString& attribute_value=element.getAttribute(XalanDOMString(name));
+	const XalanDOMString& attribute_value=element.getAttribute(*pool.transcode(name));
 	// write out result
 	r.write_no_lang(*new(pool) VString(pool.transcode(attribute_value)));
 }
@@ -177,15 +193,15 @@ static void _getAttribute(Request& r, co
 static void _setAttribute(Request& r, const String& method_name, MethodParams *params) {
 	Pool& pool=r.pool();
 	XalanElement& element=get_self_element(r, method_name);
-	const char *name=params->as_string(0, "name must not be code").cstr(String::UL_AS_IS);
-	const char *attribute_value=params->as_string(1, "value must not be code").cstr(String::UL_AS_IS);
+	const String& name=params->as_string(0, "name must be string");
+	const String& attribute_value=params->as_string(1, "value must be string");
 
 	try {
 		element.setAttribute(
-			XalanDOMString(name), 
-			XalanDOMString(attribute_value));
+			*pool.transcode(name), 
+			*pool.transcode(attribute_value));
 	} catch(const XalanDOMException& e)	{
-		pool.exception()._throw(pool, &method_name, e);
+		Exception::provide_source(pool, &method_name, e);
 	}
 }
 
@@ -193,12 +209,12 @@ static void _setAttribute(Request& r, co
 static void _removeAttribute(Request& r, const String& method_name, MethodParams *params) {
 	Pool& pool=r.pool();
 	XalanElement& element=get_self_element(r, method_name);
-	const char *name=params->as_string(0, "name must not be code").cstr(String::UL_AS_IS);
+	const String& name=params->as_string(0, "name must be string");
 
 	try {
-		element.removeAttribute(XalanDOMString(name));
+		element.removeAttribute(*pool.transcode(name));
 	} catch(const XalanDOMException& e)	{
-		pool.exception()._throw(pool, &method_name, e);
+		Exception::provide_source(pool, &method_name, e);
 	}
 }
 
@@ -206,11 +222,11 @@ static void _removeAttribute(Request& r,
 static void _getAttributeNode(Request& r, const String& method_name, MethodParams *params) {
 	Pool& pool=r.pool();
 	XalanElement& element=get_self_element(r, method_name);
-	const char *name=params->as_string(0, "name must not be code").cstr(String::UL_AS_IS);
+	const String& name=params->as_string(0, "name must be string");
 
-	if(XalanAttr *attr=element.getAttributeNode(XalanDOMString(name))) {
+	if(XalanAttr *attr=element.getAttributeNode(*pool.transcode(name))) {
 		// write out result
-		VXnode& result=*new(pool) VXnode(pool, attr);
+		VXnode& result=*new(pool) VXnode(pool, attr, false);
 		r.write_no_lang(result);
 	}
 }	
@@ -224,11 +240,11 @@ static void _setAttributeNode(Request& r
 	try {
 		if(XalanAttr *returnAttr=element.setAttributeNode(&newAttr)) {
 			// write out result
-			VXnode& result=*new(pool) VXnode(pool, returnAttr);
+			VXnode& result=*new(pool) VXnode(pool, returnAttr, false);
 			r.write_no_lang(result);
 		}
 	} catch(const XalanDOMException& e)	{
-		pool.exception()._throw(pool, &method_name, e);
+		Exception::provide_source(pool, &method_name, e);
 	}
 }	
 
@@ -239,9 +255,9 @@ static void _removeAttributeNode(Request
 	XalanAttr& oldAttr=as_attr(pool, method_name, params, 0, "oldAttr must be ATTRIBUTE node");
 
 	try {
-		/*XalanAttr *returnAttr*/element.removeAttributeNode(&oldAttr));
+		/*XalanAttr *returnAttr*/element.removeAttributeNode(&oldAttr);
 	} catch(const XalanDOMException& e)	{
-		pool.exception()._throw(pool, &method_name, e);
+		Exception::provide_source(pool, &method_name, e);
 	}
 }	
 
@@ -250,11 +266,11 @@ static void _getElementsByTagName(Reques
 	Pool& pool=r.pool();
 	XalanElement& element=get_self_element(r, method_name);
 
-	const char *name=params->as_string(0, "name must not be code").cstr(String::UL_AS_IS);
+	const String& name=params->as_string(0, "name must be string");
 
 	VHash& result=*new(pool) VHash(pool);
 	if(const XalanNodeList *nodes=
-		element.getElementsByTagName(XalanDOMString(name))) {
+		element.getElementsByTagName(*pool.transcode(name))) {
 		for(int i=0; i<nodes->getLength(); i++) {
 			String& skey=*new(pool) String(pool);
 			{
@@ -263,7 +279,7 @@ static void _getElementsByTagName(Reques
 				skey << buf;
 			}
 
-			result.hash().put(skey, new(pool) VXnode(pool, nodes->item(i)));
+			result.hash().put(skey, new(pool) VXnode(pool, nodes->item(i), false));
 		}
 	}
 
@@ -285,8 +301,8 @@ static void _select(Request& r, const St
 	VXnode& vnode=*static_cast<VXnode *>(r.self);
 
 	// expression
-	const String& expression=params->as_string(0, "expression must not be code");
-	const char *expression_cstr=expression.cstr(String::UL_AS_IS);
+	const String& expression=params->as_string(0, "expression must be string");
+	const char *expression_cstr=expression.cstr();
 	XalanDOMString dstring(expression_cstr);
 	const XalanDOMChar *expression_dcstr=dstring.c_str();
 
@@ -308,23 +324,23 @@ static void _select(Request& r, const St
 				skey << buf;
 			}
 
-			result.hash().put(skey, new(pool) VXnode(pool, list.item(i)));
+			result.hash().put(skey, new(pool) VXnode(pool, list.item(i), false));
 		}
 		result.set_name(method_name);
 		r.write_no_lang(result);
 	} catch(const XSLException& e) {
-		pool.exception()._throw(pool, &expression, e);
+		Exception::provide_source(pool, &expression, e);
 	}
 }
 
-static void _select_single(Request& r, const String& method_name, MethodParams *params) {
+static void _selectSingle(Request& r, const String& method_name, MethodParams *params) {
 //	_asm int 3;
 	Pool& pool=r.pool();
 	VXnode& vnode=*static_cast<VXnode *>(r.self);
 
 	// expression
-	const String& expression=params->as_string(0, "expression must not be code");
-	const char *expression_cstr=expression.cstr(String::UL_AS_IS);
+	const String& expression=params->as_string(0, "expression must be string");
+	const char *expression_cstr=expression.cstr();
 	XalanDOMString dstring(expression_cstr);
 	const XalanDOMChar *expression_dcstr=dstring.c_str();
 
@@ -339,12 +355,12 @@ static void _select_single(Request& r, c
 			&vnode.get_node(pool, &method_name), 
 			expression_dcstr)) {
 
-			VXnode& result=*new(pool) VXnode(pool, node);
+			VXnode& result=*new(pool) VXnode(pool, node, false);
 			result.set_name(method_name);
 			r.write_no_lang(result);
 		}
 	} catch(const XSLException& e) {
-		pool.exception()._throw(pool, &expression, e);
+		Exception::provide_source(pool, &expression, e);
 	}
 }
 
@@ -394,7 +410,7 @@ MXnode::MXnode(Pool& apool) : Methoded(a
 	add_native_method("select", Method::CT_DYNAMIC, _select, 1, 1);
 
 	// ^node.selectSingle[/some/xpath/query] = first dnode
-	add_native_method("selectSingle", Method::CT_DYNAMIC, _select_single, 1, 1);
+	add_native_method("selectSingle", Method::CT_DYNAMIC, _selectSingle, 1, 1);
 
 	// consts