removed a superfluous second sorting of the node set on xsl:copy-of, fixes

* libxslt/transform.c: removed a superfluous second sorting of
  the node set on xsl:copy-of, fixes #316288
* libxslt/xsltutils.c: marked xsltDocumentSortFunction as slow,
  obsolete and deprecated :-)
Daniel

diff --git a/ChangeLog b/ChangeLog
index c7f4975..22bcaa9 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+Wed Sep 14 14:30:03 CEST 2005 Daniel Veillard <daniel@veillard.com>
+
+       * libxslt/transform.c: removed a superfluous second sorting of
+         the node set on xsl:copy-of, fixes #316288
+       * libxslt/xsltutils.c: marked xsltDocumentSortFunction as slow,
+         obsolete and deprecated :-)
+
 Sat Sep 10 14:35:06 CEST 2005 Daniel Veillard <daniel@veillard.com>
 
        * configure.in: check environment for PYTHON, fixes #315367

diff --git a/libxslt/transform.c b/libxslt/transform.c
index 5ad625f..5bd4d40 100644 (file)
--- a/libxslt/transform.c
+++ b/libxslt/transform.c
@@ -2924,8 +2924,7 @@ xsltCopyOf(xsltTransformContextPtr ctxt, xmlNodePtr node,
 #endif
            list = res->nodesetval;
            if (list != NULL) {
-               /* sort the list in document order */
-               xsltDocumentSortFunction(list);
+               /* the list is already sorted in document order by XPath */
                /* append everything in this order under ctxt->insert */
                for (i = 0;i < list->nodeNr;i++) {
                    if (list->nodeTab[i] == NULL)

diff --git a/libxslt/xsltutils.c b/libxslt/xsltutils.c
index 5c7a4cf..af762f7 100644 (file)
--- a/libxslt/xsltutils.c
+++ b/libxslt/xsltutils.c
@@ -776,6 +776,7 @@ xsltGetQNameURI2(xsltStylesheetPtr style, xmlNodePtr node,
  * @list:  the node set
  *
  * reorder the current node list @list accordingly to the document order
+ * This function is slow, obsolete and should not be used anymore.
  */
 void
 xsltDocumentSortFunction(xmlNodeSetPtr list) {