[jboss-cvs] JBossRemoting/src/main/org/jboss/remoting/transport/socket ...

[Ron Sigal]

  User: rsigal  
  Date: 07/07/06 21:03:00

  Modified:    src/main/org/jboss/remoting/transport/socket  Tag:
                        remoting_2_x ServerThread.java
  Log:
  JBREM-706:  Test if is server side oneway invocation before ServerInvoker changes InvocationRequest.
  
  Revision  Changes    Path
  No                   revision
  
  
  No                   revision
  
  
  1.29.2.23 +3 -2      JBossRemoting/src/main/org/jboss/remoting/transport/socket/ServerThread.java
  
  (In the diff below, changes in quantity of whitespace are not shown.)
  
  Index: ServerThread.java
  ===================================================================
  RCS file: /cvsroot/jboss/JBossRemoting/src/main/org/jboss/remoting/transport/socket/ServerThread.java,v
  retrieving revision 1.29.2.22
  retrieving revision 1.29.2.23
  diff -u -b -r1.29.2.22 -r1.29.2.23
  --- ServerThread.java	6 Jul 2007 06:15:01 -0000	1.29.2.22
  +++ ServerThread.java	7 Jul 2007 01:03:00 -0000	1.29.2.23
  @@ -67,7 +67,7 @@
    * @author <a href="mailto:tom at jboss.org">Tom Elrod</a>
    * @author <a href="mailto:ovidiu at jboss.org">Ovidiu Feodorov</a>
    *
  - * @version $Revision: 1.29.2.22 $
  + * @version $Revision: 1.29.2.23 $
    */
   public class ServerThread extends Thread
   {
  @@ -548,6 +548,7 @@
            performVersioning = false;
         }
   
  +      boolean isServerSideOnewayRequest = isServerSideOneway(req);
         Object resp = null;
   
         try
  @@ -571,7 +572,7 @@
   
         Thread.interrupted(); // clear interrupted state so we don't fail on socket writes
   
  -      if(isServerSideOneway(req))
  +      if(isServerSideOnewayRequest)
         {
            if(trace) { log.trace("oneway request, writing no reply on the wire"); }
         }