You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commons-dev@ws.apache.org by ve...@apache.org on 2008/12/09 21:17:15 UTC

svn commit: r724860 - /webservices/commons/trunk/modules/axiom/modules/axiom-api/src/main/java/org/apache/axiom/om/impl/builder/SAXOMBuilder.java

Author: veithen
Date: Tue Dec  9 12:17:15 2008
New Revision: 724860

URL: http://svn.apache.org/viewvc?rev=724860&view=rev
Log:
WSCOMMONS-327: Added a note related to a recent change in SAXOMBuilder.

Modified:
    webservices/commons/trunk/modules/axiom/modules/axiom-api/src/main/java/org/apache/axiom/om/impl/builder/SAXOMBuilder.java

Modified: webservices/commons/trunk/modules/axiom/modules/axiom-api/src/main/java/org/apache/axiom/om/impl/builder/SAXOMBuilder.java
URL: http://svn.apache.org/viewvc/webservices/commons/trunk/modules/axiom/modules/axiom-api/src/main/java/org/apache/axiom/om/impl/builder/SAXOMBuilder.java?rev=724860&r1=724859&r2=724860&view=diff
==============================================================================
--- webservices/commons/trunk/modules/axiom/modules/axiom-api/src/main/java/org/apache/axiom/om/impl/builder/SAXOMBuilder.java (original)
+++ webservices/commons/trunk/modules/axiom/modules/axiom-api/src/main/java/org/apache/axiom/om/impl/builder/SAXOMBuilder.java Tue Dec  9 12:17:15 2008
@@ -121,6 +121,10 @@
         for (int i = 0; i < j; i++) {
             // Note that some SAX parsers report namespace declarations as attributes in addition
             // to calling start/endPrefixMapping.
+            // NOTE: This filter was introduced to make SAXOMBuilder work with some versions of
+            //       XMLBeans (2.3.0). It is not clear whether this is a bug in XMLBeans or not.
+            //       See http://forum.springframework.org/showthread.php?t=43958 for a discussion.
+            //       If this test causes problems with other parsers, don't hesitate to remove it.
             if (!atts.getQName(i).startsWith("xmlns")) {
                 OMAttribute attr = nextElem.addAttribute(atts.getLocalName(i), atts.getValue(i),
                         nextElem.findNamespace(atts.getURI(i), null));