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 sc...@apache.org on 2008/04/15 21:57:13 UTC

svn commit: r648395 - in /webservices/commons/trunk/modules/axiom/modules: axiom-api/src/main/java/org/apache/axiom/om/OMXMLParserWrapper.java axiom-impl/src/main/java/org/apache/axiom/om/impl/llom/OMStAXWrapper.java

Author: scheu
Date: Tue Apr 15 12:57:09 2008
New Revision: 648395

URL: http://svn.apache.org/viewvc?rev=648395&view=rev
Log:
WSCOMMONS-321
Contributor:Rich Scheuerle
Root cause of for NoSuchElementException (found by JAXWS CTS testing)
OMStAXWrapper code was accidentally disabling caching.

The validating test is in Axis2/JAXWS and will be contributed soon.

Modified:
    webservices/commons/trunk/modules/axiom/modules/axiom-api/src/main/java/org/apache/axiom/om/OMXMLParserWrapper.java
    webservices/commons/trunk/modules/axiom/modules/axiom-impl/src/main/java/org/apache/axiom/om/impl/llom/OMStAXWrapper.java

Modified: webservices/commons/trunk/modules/axiom/modules/axiom-api/src/main/java/org/apache/axiom/om/OMXMLParserWrapper.java
URL: http://svn.apache.org/viewvc/webservices/commons/trunk/modules/axiom/modules/axiom-api/src/main/java/org/apache/axiom/om/OMXMLParserWrapper.java?rev=648395&r1=648394&r2=648395&view=diff
==============================================================================
--- webservices/commons/trunk/modules/axiom/modules/axiom-api/src/main/java/org/apache/axiom/om/OMXMLParserWrapper.java (original)
+++ webservices/commons/trunk/modules/axiom/modules/axiom-api/src/main/java/org/apache/axiom/om/OMXMLParserWrapper.java Tue Apr 15 12:57:09 2008
@@ -48,6 +48,11 @@
      * @throws OMException
      */
     void setCache(boolean b) throws OMException;
+    
+    /**
+     * @return true if caching
+     */
+    boolean isCache();
 
     /**
      * Allows to access the underlying parser. Since the parser depends on the underlying

Modified: webservices/commons/trunk/modules/axiom/modules/axiom-impl/src/main/java/org/apache/axiom/om/impl/llom/OMStAXWrapper.java
URL: http://svn.apache.org/viewvc/webservices/commons/trunk/modules/axiom/modules/axiom-impl/src/main/java/org/apache/axiom/om/impl/llom/OMStAXWrapper.java?rev=648395&r1=648394&r2=648395&view=diff
==============================================================================
--- webservices/commons/trunk/modules/axiom/modules/axiom-impl/src/main/java/org/apache/axiom/om/impl/llom/OMStAXWrapper.java (original)
+++ webservices/commons/trunk/modules/axiom/modules/axiom-impl/src/main/java/org/apache/axiom/om/impl/llom/OMStAXWrapper.java Tue Apr 15 12:57:09 2008
@@ -209,8 +209,11 @@
         try {
             if (startNode instanceof OMSourcedElement && 
                     !cache && builder != null) {
+                if (!builder.isCache()) {
+                    resetCache = true;
+                }
                 builder.setCache(true); // bootstrap the navigator
-                resetCache = true;
+                
             }
         } catch(Throwable t) {}
         



---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@ws.apache.org
For additional commands, e-mail: commons-dev-help@ws.apache.org