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 2009/12/22 11:47:30 UTC

svn commit: r893133 - in /webservices/commons/trunk/modules/axiom/modules: axiom-api/src/main/java/org/apache/axiom/om/ axiom-dom/src/main/java/org/apache/axiom/om/impl/dom/ axiom-impl/src/main/java/org/apache/axiom/om/impl/llom/

Author: veithen
Date: Tue Dec 22 10:47:24 2009
New Revision: 893133

URL: http://svn.apache.org/viewvc?rev=893133&view=rev
Log:
Removed the throws declaration from OMElement#getNamespace(). This method never throws OMException. Note that since OMException is an unchecked exception, this change only impacts the documentation, not the runtime behavior.

Modified:
    webservices/commons/trunk/modules/axiom/modules/axiom-api/src/main/java/org/apache/axiom/om/OMElement.java
    webservices/commons/trunk/modules/axiom/modules/axiom-dom/src/main/java/org/apache/axiom/om/impl/dom/ElementImpl.java
    webservices/commons/trunk/modules/axiom/modules/axiom-impl/src/main/java/org/apache/axiom/om/impl/llom/OMElementImpl.java

Modified: webservices/commons/trunk/modules/axiom/modules/axiom-api/src/main/java/org/apache/axiom/om/OMElement.java
URL: http://svn.apache.org/viewvc/webservices/commons/trunk/modules/axiom/modules/axiom-api/src/main/java/org/apache/axiom/om/OMElement.java?rev=893133&r1=893132&r2=893133&view=diff
==============================================================================
--- webservices/commons/trunk/modules/axiom/modules/axiom-api/src/main/java/org/apache/axiom/om/OMElement.java (original)
+++ webservices/commons/trunk/modules/axiom/modules/axiom-api/src/main/java/org/apache/axiom/om/OMElement.java Tue Dec 22 10:47:24 2009
@@ -400,9 +400,8 @@
 
     /**
      * @return Returns the OMNamespace object associated with this element
-     * @throws OMException
      */
-    OMNamespace getNamespace() throws OMException;
+    OMNamespace getNamespace();
 
     /**
      * Sets the Namespace. This will first search for a namespace in the current scope with the

Modified: webservices/commons/trunk/modules/axiom/modules/axiom-dom/src/main/java/org/apache/axiom/om/impl/dom/ElementImpl.java
URL: http://svn.apache.org/viewvc/webservices/commons/trunk/modules/axiom/modules/axiom-dom/src/main/java/org/apache/axiom/om/impl/dom/ElementImpl.java?rev=893133&r1=893132&r2=893133&view=diff
==============================================================================
--- webservices/commons/trunk/modules/axiom/modules/axiom-dom/src/main/java/org/apache/axiom/om/impl/dom/ElementImpl.java (original)
+++ webservices/commons/trunk/modules/axiom/modules/axiom-dom/src/main/java/org/apache/axiom/om/impl/dom/ElementImpl.java Tue Dec 22 10:47:24 2009
@@ -943,7 +943,7 @@
      *
      * @see org.apache.axiom.om.OMElement#getNamespace()
      */
-    public OMNamespace getNamespace() throws OMException {
+    public OMNamespace getNamespace() {
         return namespace != null ? namespace : getDefaultNamespace();
     }
 

Modified: webservices/commons/trunk/modules/axiom/modules/axiom-impl/src/main/java/org/apache/axiom/om/impl/llom/OMElementImpl.java
URL: http://svn.apache.org/viewvc/webservices/commons/trunk/modules/axiom/modules/axiom-impl/src/main/java/org/apache/axiom/om/impl/llom/OMElementImpl.java?rev=893133&r1=893132&r2=893133&view=diff
==============================================================================
--- webservices/commons/trunk/modules/axiom/modules/axiom-impl/src/main/java/org/apache/axiom/om/impl/llom/OMElementImpl.java (original)
+++ webservices/commons/trunk/modules/axiom/modules/axiom-impl/src/main/java/org/apache/axiom/om/impl/llom/OMElementImpl.java Tue Dec 22 10:47:24 2009
@@ -1014,12 +1014,7 @@
         this.qName = null;
     }
 
-    /**
-     * Method getNamespace.
-     *
-     * @throws OMException
-     */
-    public OMNamespace getNamespace() throws OMException {
+    public OMNamespace getNamespace() {
 //        return ns != null ? ns : DEFAULT_DEFAULT_NS_OBJECT;
         if (ns == null) {
             // User wants to keep this element in the default default namespace. Let's try to see the default namespace