You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tuscany.apache.org by an...@apache.org on 2010/07/20 14:09:58 UTC

svn commit: r965821 - /tuscany/sca-java-2.x/trunk/modules/assembly/src/main/java/org/apache/tuscany/sca/interfacedef/impl/InterfaceContractMapperImpl.java

Author: antelder
Date: Tue Jul 20 12:09:58 2010
New Revision: 965821

URL: http://svn.apache.org/viewvc?rev=965821&view=rev
Log:
Add a check for both source and target operations matching being oneway or not

Modified:
    tuscany/sca-java-2.x/trunk/modules/assembly/src/main/java/org/apache/tuscany/sca/interfacedef/impl/InterfaceContractMapperImpl.java

Modified: tuscany/sca-java-2.x/trunk/modules/assembly/src/main/java/org/apache/tuscany/sca/interfacedef/impl/InterfaceContractMapperImpl.java
URL: http://svn.apache.org/viewvc/tuscany/sca-java-2.x/trunk/modules/assembly/src/main/java/org/apache/tuscany/sca/interfacedef/impl/InterfaceContractMapperImpl.java?rev=965821&r1=965820&r2=965821&view=diff
==============================================================================
--- tuscany/sca-java-2.x/trunk/modules/assembly/src/main/java/org/apache/tuscany/sca/interfacedef/impl/InterfaceContractMapperImpl.java (original)
+++ tuscany/sca-java-2.x/trunk/modules/assembly/src/main/java/org/apache/tuscany/sca/interfacedef/impl/InterfaceContractMapperImpl.java Tue Jul 20 12:09:58 2010
@@ -229,6 +229,16 @@ public class InterfaceContractMapperImpl
             return false;
         }
 
+        if (source.isNonBlocking() != target.isNonBlocking()) {
+            if (audit != null){
+                audit.append("operations one-way not the same, source = " +
+                             source.isNonBlocking() + 
+                             " target = " +
+                             target.isNonBlocking());
+            }            
+            return false;
+        }
+
         boolean passByValue = (source.getInterface().isRemotable()) && byValue;
 
         //        if (source.getInterface().isRemotable()) {