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

[Ron Sigal]

  User: rsigal  
  Date: 07/01/31 03:58:20

  Modified:    src/main/org/jboss/remoting/transport/socket  Tag:
                        remoting_2_x MicroSocketClientInvoker.java
  Log:
  JBREM-598:  Added comment in transport().
  
  Revision  Changes    Path
  No                   revision
  
  
  No                   revision
  
  
  1.16.2.20 +5 -1      JBossRemoting/src/main/org/jboss/remoting/transport/socket/MicroSocketClientInvoker.java
  
  (In the diff below, changes in quantity of whitespace are not shown.)
  
  Index: MicroSocketClientInvoker.java
  ===================================================================
  RCS file: /cvsroot/jboss/JBossRemoting/src/main/org/jboss/remoting/transport/socket/MicroSocketClientInvoker.java,v
  retrieving revision 1.16.2.19
  retrieving revision 1.16.2.20
  diff -u -b -r1.16.2.19 -r1.16.2.20
  --- MicroSocketClientInvoker.java	31 Jan 2007 08:53:31 -0000	1.16.2.19
  +++ MicroSocketClientInvoker.java	31 Jan 2007 08:58:20 -0000	1.16.2.20
  @@ -35,7 +35,7 @@
    * @author <a href="mailto:telrod at e2technologies.net">Tom Elrod</a>
    * @author <a href="mailto:ovidiu at jboss.org">Ovidiu Feodorov</a>
    *
  - * @version $Revision: 1.16.2.19 $
  + * @version $Revision: 1.16.2.20 $
    */
   public class MicroSocketClientInvoker extends RemoteClientInvoker
   {
  @@ -562,6 +562,10 @@
               end = System.currentTimeMillis() - start;
               readTime += end;
   
  +            // Note that resetting the timeout value after closing the socket results
  +            // in an exception, so the reset is not done in a finally clause.  However,
  +            // if a catch clause is ever added that does not close the socket, care
  +            // must be taken to reset the timeout in that case.
               if (tempTimeout >= 0)
               {
                  socketWrapper.setTimeout(savedTimeout);