You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tuscany.apache.org by jm...@apache.org on 2006/11/23 08:02:11 UTC

svn commit: r478494 - in /incubator/tuscany/java/sca/kernel: core/src/main/java/org/apache/tuscany/core/wire/jdk/ spi/src/main/java/org/apache/tuscany/spi/wire/ spi/src/test/java/org/apache/tuscany/spi/wire/

Author: jmarino
Date: Wed Nov 22 23:02:10 2006
New Revision: 478494

URL: http://svn.apache.org/viewvc?view=rev&rev=478494
Log:
remove unsafe storage of message id on JDKInboundInvocationHandler; remove related unused methods on AbstractInboundInvocationHandler

Modified:
    incubator/tuscany/java/sca/kernel/core/src/main/java/org/apache/tuscany/core/wire/jdk/JDKInboundInvocationHandler.java
    incubator/tuscany/java/sca/kernel/spi/src/main/java/org/apache/tuscany/spi/wire/AbstractInboundInvocationHandler.java
    incubator/tuscany/java/sca/kernel/spi/src/test/java/org/apache/tuscany/spi/wire/AbstractInboundInvocationHandlerTestCase.java

Modified: incubator/tuscany/java/sca/kernel/core/src/main/java/org/apache/tuscany/core/wire/jdk/JDKInboundInvocationHandler.java
URL: http://svn.apache.org/viewvc/incubator/tuscany/java/sca/kernel/core/src/main/java/org/apache/tuscany/core/wire/jdk/JDKInboundInvocationHandler.java?view=diff&rev=478494&r1=478493&r2=478494
==============================================================================
--- incubator/tuscany/java/sca/kernel/core/src/main/java/org/apache/tuscany/core/wire/jdk/JDKInboundInvocationHandler.java (original)
+++ incubator/tuscany/java/sca/kernel/core/src/main/java/org/apache/tuscany/core/wire/jdk/JDKInboundInvocationHandler.java Wed Nov 22 23:02:10 2006
@@ -49,8 +49,6 @@
      */
     private Map<Method, ChainHolder> chains;
     private WorkContext context;
-    private Object messageId;
-    private Object correlationId;
 
     public JDKInboundInvocationHandler(Map<Method, InboundInvocationChain> invocationChains, WorkContext context) {
         this.chains = new HashMap<Method, ChainHolder>(invocationChains.size());
@@ -101,9 +99,7 @@
             assert chain != null;
             invoker = chain.getTargetInvoker();
         }
-        messageId = context.getCurrentMessageId();
         context.setCurrentMessageId(null);
-        correlationId = context.getCurrentCorrelationId();
         context.setCurrentCorrelationId(null);
         return invoke(chain, invoker, args);
     }
@@ -126,14 +122,6 @@
             this.chain = config;
         }
 
-    }
-
-    protected Object getMessageId() {
-        return messageId;
-    }
-
-    protected Object getCorrelationId() {
-        return correlationId;
     }
 
 }

Modified: incubator/tuscany/java/sca/kernel/spi/src/main/java/org/apache/tuscany/spi/wire/AbstractInboundInvocationHandler.java
URL: http://svn.apache.org/viewvc/incubator/tuscany/java/sca/kernel/spi/src/main/java/org/apache/tuscany/spi/wire/AbstractInboundInvocationHandler.java?view=diff&rev=478494&r1=478493&r2=478494
==============================================================================
--- incubator/tuscany/java/sca/kernel/spi/src/main/java/org/apache/tuscany/spi/wire/AbstractInboundInvocationHandler.java (original)
+++ incubator/tuscany/java/sca/kernel/spi/src/main/java/org/apache/tuscany/spi/wire/AbstractInboundInvocationHandler.java Wed Nov 22 23:02:10 2006
@@ -59,7 +59,4 @@
         }
     }
 
-    protected abstract Object getMessageId();
-
-    protected abstract Object getCorrelationId();
 }

Modified: incubator/tuscany/java/sca/kernel/spi/src/test/java/org/apache/tuscany/spi/wire/AbstractInboundInvocationHandlerTestCase.java
URL: http://svn.apache.org/viewvc/incubator/tuscany/java/sca/kernel/spi/src/test/java/org/apache/tuscany/spi/wire/AbstractInboundInvocationHandlerTestCase.java?view=diff&rev=478494&r1=478493&r2=478494
==============================================================================
--- incubator/tuscany/java/sca/kernel/spi/src/test/java/org/apache/tuscany/spi/wire/AbstractInboundInvocationHandlerTestCase.java (original)
+++ incubator/tuscany/java/sca/kernel/spi/src/test/java/org/apache/tuscany/spi/wire/AbstractInboundInvocationHandlerTestCase.java Wed Nov 22 23:02:10 2006
@@ -25,13 +25,6 @@
 
     private class InvocationHandler extends AbstractInboundInvocationHandler {
 
-        protected Object getMessageId() {
-            return new Object();
-        }
-
-        protected Object getCorrelationId() {
-            return new Object();
-        }
     }
 
     private class MockInterceptor implements Interceptor {



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