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/05/19 21:02:51 UTC

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

Author: veithen
Date: Tue May 19 19:02:50 2009
New Revision: 776415

URL: http://svn.apache.org/viewvc?rev=776415&view=rev
Log:
Added a review comment.

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

Modified: webservices/commons/trunk/modules/axiom/modules/axiom-api/src/main/java/org/apache/axiom/om/impl/OMStAXWrapper.java
URL: http://svn.apache.org/viewvc/webservices/commons/trunk/modules/axiom/modules/axiom-api/src/main/java/org/apache/axiom/om/impl/OMStAXWrapper.java?rev=776415&r1=776414&r2=776415&view=diff
==============================================================================
--- webservices/commons/trunk/modules/axiom/modules/axiom-api/src/main/java/org/apache/axiom/om/impl/OMStAXWrapper.java (original)
+++ webservices/commons/trunk/modules/axiom/modules/axiom-api/src/main/java/org/apache/axiom/om/impl/OMStAXWrapper.java Tue May 19 19:02:50 2009
@@ -1549,6 +1549,9 @@
             lastNode != null && 
             lastNode instanceof OMText) {
             OMText text = (OMText) lastNode;
+            // TODO: this is in contradiction with the Javadoc
+            //       which says that blobcid is the content ID without the surrounding
+            //       angle brackets and "cid:" prefix
             if (text.isOptimized() &&
                     blobcid.equals("cid:" + text.getContentID())) {
                dh = (DataHandler) text.getDataHandler();