You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@ws.apache.org by ve...@apache.org on 2012/09/05 21:21:59 UTC

svn commit: r1381309 - /webservices/commons/trunk/modules/axiom/modules/axiom-dom/src/main/java/org/apache/axiom/om/impl/dom/AttrImpl.java

Author: veithen
Date: Wed Sep  5 19:21:59 2012
New Revision: 1381309

URL: http://svn.apache.org/viewvc?rev=1381309&view=rev
Log:
Added TODO item.

Modified:
    webservices/commons/trunk/modules/axiom/modules/axiom-dom/src/main/java/org/apache/axiom/om/impl/dom/AttrImpl.java

Modified: webservices/commons/trunk/modules/axiom/modules/axiom-dom/src/main/java/org/apache/axiom/om/impl/dom/AttrImpl.java
URL: http://svn.apache.org/viewvc/webservices/commons/trunk/modules/axiom/modules/axiom-dom/src/main/java/org/apache/axiom/om/impl/dom/AttrImpl.java?rev=1381309&r1=1381308&r2=1381309&view=diff
==============================================================================
--- webservices/commons/trunk/modules/axiom/modules/axiom-dom/src/main/java/org/apache/axiom/om/impl/dom/AttrImpl.java (original)
+++ webservices/commons/trunk/modules/axiom/modules/axiom-dom/src/main/java/org/apache/axiom/om/impl/dom/AttrImpl.java Wed Sep  5 19:21:59 2012
@@ -110,6 +110,7 @@ public class AttrImpl extends RootNode i
         this.localName = name;
         //If this is a default namespace attr
         if (XMLConstants.XMLNS_ATTRIBUTE.equals(name)) {
+            // TODO: this looks wrong; if the attribute name is "xmlns", then the prefix shouldn't be "xmlns"
             this.namespace = new OMNamespaceImpl(
                     XMLConstants.XMLNS_ATTRIBUTE_NS_URI, XMLConstants.XMLNS_ATTRIBUTE);
         }