You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tuscany.apache.org by rf...@apache.org on 2007/02/22 06:19:33 UTC

svn commit: r510402 - /incubator/tuscany/branches/sca-java-integration/sca/kernel/core/src/main/java/org/apache/tuscany/core/databinding/impl/DataBindingJavaInterfaceProcessor.java

Author: rfeng
Date: Wed Feb 21 21:19:28 2007
New Revision: 510402

URL: http://svn.apache.org/viewvc?view=rev&rev=510402
Log:
Check in a workaround to bypass a bug in IBM JDK 5.0

Modified:
    incubator/tuscany/branches/sca-java-integration/sca/kernel/core/src/main/java/org/apache/tuscany/core/databinding/impl/DataBindingJavaInterfaceProcessor.java

Modified: incubator/tuscany/branches/sca-java-integration/sca/kernel/core/src/main/java/org/apache/tuscany/core/databinding/impl/DataBindingJavaInterfaceProcessor.java
URL: http://svn.apache.org/viewvc/incubator/tuscany/branches/sca-java-integration/sca/kernel/core/src/main/java/org/apache/tuscany/core/databinding/impl/DataBindingJavaInterfaceProcessor.java?view=diff&rev=510402&r1=510401&r2=510402
==============================================================================
--- incubator/tuscany/branches/sca-java-integration/sca/kernel/core/src/main/java/org/apache/tuscany/core/databinding/impl/DataBindingJavaInterfaceProcessor.java (original)
+++ incubator/tuscany/branches/sca-java-integration/sca/kernel/core/src/main/java/org/apache/tuscany/core/databinding/impl/DataBindingJavaInterfaceProcessor.java Wed Feb 21 21:19:28 2007
@@ -113,7 +113,18 @@
         Class cls = (Class) dataType.getPhysical();
         if (cls.isPrimitive() || SIMPLE_TYPE_SET.contains(cls)) {
             dataType.setDataBinding(SIMPLE_JAVA_OBJECTS);
-        } else if (cls == String.class && (dataBinding == null || !dataBinding.equals(String.class.getName()))) {
+        } 
+        /**
+         * [rfeng] The following code hits a bug in IBM JDK 5.0          
+         * if (cls == String.class && (dataBinding == null || !dataBinding.equals(String.class.getName()))) {
+         *      // Identify the String as a simple type
+         *      dataType.setDataBinding(SIMPLE_JAVA_OBJECTS);
+         * }     
+         */      
+        
+        boolean plainString =
+            cls == String.class && ((dataBinding == null) || !dataBinding.equals(String.class.getName()));
+        if (plainString) {
             // Identify the String as a simple type
             dataType.setDataBinding(SIMPLE_JAVA_OBJECTS);
         }



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