]> granicus.if.org Git - php/commitdiff
optimize sxe_add_registered_namespaces
authorRob Richards <rrichards@php.net>
Tue, 1 Nov 2005 23:21:29 +0000 (23:21 +0000)
committerRob Richards <rrichards@php.net>
Tue, 1 Nov 2005 23:21:29 +0000 (23:21 +0000)
add only first encountered prefixes to namespace arrays
update test

ext/simplexml/simplexml.c
ext/simplexml/tests/025.phpt

index f9bc29d5dd39b5cd3b957347d5a4ff0460f6128f..67204e2fd1d5a17a34051c9538ecd782c888174b 100644 (file)
@@ -1106,7 +1106,10 @@ SXE_METHOD(asXML)
 
 static inline void sxe_add_namespace_name(zval *return_value, xmlNsPtr ns)
 {
-       add_assoc_string(return_value, SXE_NS_PREFIX(ns), (char*)ns->href, 1);
+       char *prefix = SXE_NS_PREFIX(ns);
+       if (zend_hash_exists(Z_ARRVAL_P(return_value), prefix, strlen(prefix) + 1) == 0) {
+               add_assoc_string(return_value, prefix, (char*)ns->href, 1);
+       }
 }
 
 static void sxe_add_namespaces(php_sxe_object *sxe, xmlNodePtr node, zend_bool recursive, zval *return_value TSRMLS_DC) /* {{{ */
@@ -1167,25 +1170,27 @@ next_iter:
 }
 /* }}} */
 
-static void sxe_add_registered_namespaces(php_sxe_object *sxe, xmlDocPtr doc, xmlNodePtr node, zend_bool recursive, zval *return_value TSRMLS_DC) /* {{{ */
+static void sxe_add_registered_namespaces(php_sxe_object *sxe, xmlNodePtr node, zend_bool recursive, zval *return_value TSRMLS_DC) /* {{{ */
 {
-       xmlNsPtr *ns = xmlGetNsList(doc, node);
-
-       while (ns && ns[0]) {
-               sxe_add_namespace_name(return_value, ns[0]);
-               ns++;
-       }
+       xmlNsPtr ns;
 
-       if (recursive) {
-               node = node->children;
-               while (node) {
-                       sxe_add_registered_namespaces(sxe, doc, node, recursive, return_value TSRMLS_CC);
-                       node = node->next;
+       if (node->type == XML_ELEMENT_NODE) {
+               ns = node->nsDef;
+               while (ns != NULL) {
+                       sxe_add_namespace_name(return_value, ns);
+                       ns = ns->next;
+               }
+               if (recursive) {
+                       node = node->children;
+                       while (node) {
+                               sxe_add_registered_namespaces(sxe, node, recursive, return_value TSRMLS_CC);
+                               node = node->next;
+                       }
                }
        }
 }
 
-/* {{{ proto string SimpleXMLElement::getDocNamespaces([bool recursve])
+/* {{{ proto string SimpleXMLElement::getDocNamespaces([bool recursive])
    Return all namespaces registered with document */
 SXE_METHOD(getDocNamespaces)
 {
@@ -1200,7 +1205,7 @@ SXE_METHOD(getDocNamespaces)
 
        sxe = php_sxe_fetch_object(getThis() TSRMLS_CC);
 
-       sxe_add_registered_namespaces(sxe, (xmlDocPtr)sxe->document->ptr, xmlDocGetRootElement((xmlDocPtr)sxe->document->ptr), recursive, return_value TSRMLS_CC);
+       sxe_add_registered_namespaces(sxe, xmlDocGetRootElement((xmlDocPtr)sxe->document->ptr), recursive, return_value TSRMLS_CC);
 }
 /* }}} */
 
index 5eb873a1fa875977301f93066420ed33b7ddb334..8579a1e09f272dcf643712a92a58166fdf390243 100755 (executable)
@@ -8,7 +8,7 @@ SimpleXML: getting namespaces
 $xml =<<<EOF
 <?xml version='1.0'?>
 <xhtml:html xmlns:html='http://www.w3.org/1999/xhtml' xmlns:xhtml='http://www.w3.org/TR/REC-html40'>
-<xhtml:head><xhtml:title>bla</xhtml:title></xhtml:head>
+<xhtml:head><xhtml:title xmlns:xhtml='http://www.w3.org/TR/REC-html401'>bla</xhtml:title></xhtml:head>
 <xhtml:body html:title="b">
 <html:h1>bla</html:h1>
 <foo:bar xmlns:foo='foobar' xmlns:baz='foobarbaz'/>
@@ -25,7 +25,9 @@ var_dump($sxe->getDocNamespaces(true));
 
 $xml =<<<EOF
 <?xml version='1.0'?>
-<html xmlns='http://www.w3.org/1999/xhtml'/>
+<html xmlns='http://www.w3.org/1999/xhtml'>
+<head><title xmlns='http://www.w3.org/TR/REC-html40'>bla</title></head>
+</html>
 EOF;
 
 $sxe = simplexml_load_string($xml);