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 ch...@apache.org on 2006/05/02 07:36:30 UTC

svn commit: r398815 - in /webservices/commons/trunk/modules/axiom/src/org/apache/axiom/om: OMElement.java OMFactory.java impl/llom/OMElementImpl.java

Author: chinthaka
Date: Mon May  1 22:36:19 2006
New Revision: 398815

URL: http://svn.apache.org/viewcvs?rev=398815&view=rev
Log:
Improving java docs

Modified:
    webservices/commons/trunk/modules/axiom/src/org/apache/axiom/om/OMElement.java
    webservices/commons/trunk/modules/axiom/src/org/apache/axiom/om/OMFactory.java
    webservices/commons/trunk/modules/axiom/src/org/apache/axiom/om/impl/llom/OMElementImpl.java

Modified: webservices/commons/trunk/modules/axiom/src/org/apache/axiom/om/OMElement.java
URL: http://svn.apache.org/viewcvs/webservices/commons/trunk/modules/axiom/src/org/apache/axiom/om/OMElement.java?rev=398815&r1=398814&r2=398815&view=diff
==============================================================================
--- webservices/commons/trunk/modules/axiom/src/org/apache/axiom/om/OMElement.java (original)
+++ webservices/commons/trunk/modules/axiom/src/org/apache/axiom/om/OMElement.java Mon May  1 22:36:19 2006
@@ -46,7 +46,7 @@
      *               caller is expected to ensure that the URI is a valid namespace name.
      * @param prefix The prefix to associate with the given namespace.
      *               The caller is expected to ensure that this is a valid XML prefix.
-     *               If null is given, first this will check for an existing namespace
+     *               If "" is given, first this will check for an existing namespace
      *               with the same uri. If not found, a prefix will be auto-generated.
      * @return Returns the created namespace information item.
      * @see #declareNamespace(OMNamespace)
@@ -246,6 +246,11 @@
      * @return Returns a String representing the concatenation of the child text nodes.
      */
     public String getText();
+
+    /**
+     * OMText can contain its information as a QName as well. This will return the text as a QName 
+     * @return
+     */
     public QName getTextAsQName();
 
     /**

Modified: webservices/commons/trunk/modules/axiom/src/org/apache/axiom/om/OMFactory.java
URL: http://svn.apache.org/viewcvs/webservices/commons/trunk/modules/axiom/src/org/apache/axiom/om/OMFactory.java?rev=398815&r1=398814&r2=398815&view=diff
==============================================================================
--- webservices/commons/trunk/modules/axiom/src/org/apache/axiom/om/OMFactory.java (original)
+++ webservices/commons/trunk/modules/axiom/src/org/apache/axiom/om/OMFactory.java Mon May  1 22:36:19 2006
@@ -33,7 +33,7 @@
 
     /**
      * @param localName
-     * @param ns
+     * @param ns - This can be null
      */
     public OMElement createOMElement(String localName, OMNamespace ns);
 
@@ -41,7 +41,7 @@
 
     /**
      * @param localName
-     * @param ns
+     * @param ns - this can be null
      * @param parent
      * @param builder
      */

Modified: webservices/commons/trunk/modules/axiom/src/org/apache/axiom/om/impl/llom/OMElementImpl.java
URL: http://svn.apache.org/viewcvs/webservices/commons/trunk/modules/axiom/src/org/apache/axiom/om/impl/llom/OMElementImpl.java?rev=398815&r1=398814&r2=398815&view=diff
==============================================================================
--- webservices/commons/trunk/modules/axiom/src/org/apache/axiom/om/impl/llom/OMElementImpl.java (original)
+++ webservices/commons/trunk/modules/axiom/src/org/apache/axiom/om/impl/llom/OMElementImpl.java Mon May  1 22:36:19 2006
@@ -305,6 +305,9 @@
      * @param uri
      */
     public OMNamespace declareDefaultNamespace(String uri) {
+        if (uri == null || "".equals(uri)) {
+            return null;
+        }
         OMNamespaceImpl namespace = new OMNamespaceImpl(uri, "");
 
         if (namespaces == null) {