TUSCANY-3916 - Correct comment to reflect what I believe is the reality

git-svn-id: http://svn.us.apache.org/repos/asf/tuscany@1159624 13f79535-47bb-0310-9956-ffa450edef68

1 files changed, 4 insertions, 3 deletions

diff --git a/sca-java-2.x/trunk/modules/assembly/src/main/java/org/apache/tuscany/sca/interfacedef/impl/InterfaceContractMapperImpl.java b/sca-java-2.x/trunk/modules/assembly/src/main/java/org/apache/tuscany/sca/interfacedef/impl/InterfaceContractMapperImpl.java
index b444c4bd3f..17b41c30a2 100644
--- a/sca-java-2.x/trunk/modules/assembly/src/main/java/org/apache/tuscany/sca/interfacedef/impl/InterfaceContractMapperImpl.java
+++ b/sca-java-2.x/trunk/modules/assembly/src/main/java/org/apache/tuscany/sca/interfacedef/impl/InterfaceContractMapperImpl.java
@@ -90,9 +90,10 @@ public class InterfaceContractMapperImpl implements InterfaceContractMapper {
             }            
             targetLogicalType = (XMLType)target.getLogical();
 
-            // The logical type seems to be null in some cases, e.g. when the 
-            // argument or return type is something like a Map. 
-            // TODO - check when some type give rise to a null logical type
+            // The logical type is null in some cases. This is when the 
+            // runtime can't determine the XML type for a particular type, for
+            // example for a non-JAXB Java bean. This makes interface checking
+            // rather lenient with errors being detected at runtime
             if (sourceLogicalType.getTypeName() == null ||
                 targetLogicalType.getTypeName() == null) {
                 return true;