cvs commit: jakarta-tomcat-connectors/coyote/src/java/org/apache/coyote/tomcat3 Tomcat3Request.java

[billbarker]

billbarker    02/04/09 00:02:53

  Modified:    coyote/src/java/org/apache/coyote/tomcat3
                        Tomcat3Request.java
  Log:
  Tie into the backend remoteAddr & remoteHost lookup.
  
  The duplicates are ugly, but until we can safely stop the agressive recycling, they 
provide huge performance benifits by not doing the lookup on a per-request basis.
  
  Revision  Changes    Path
  1.5       +22 -0     
jakarta-tomcat-connectors/coyote/src/java/org/apache/coyote/tomcat3/Tomcat3Request.java
  
  Index: Tomcat3Request.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-tomcat-connectors/coyote/src/java/org/apache/coyote/tomcat3/Tomcat3Request.java,v
  retrieving revision 1.4
  retrieving revision 1.5
  diff -u -r1.4 -r1.5
  --- Tomcat3Request.java       8 Apr 2002 23:47:04 -0000       1.4
  +++ Tomcat3Request.java       9 Apr 2002 07:02:53 -0000       1.5
  @@ -202,14 +202,36 @@
   
       public MessageBytes remoteAddr() {
        if( remoteAddrMB.isNull() ) {
  +         try {
  +             remoteAddrMB.duplicate(coyoteRequest.remoteAddr());
  +         } catch(IOException iex) { // can't happen
  +         }
  +     }
  +     if( remoteAddrMB.isNull() ) {
            coyoteRequest.action( ActionCode.ACTION_REQ_HOST_ATTRIBUTE, coyoteRequest 
);
  +         try{
  +             remoteAddrMB.duplicate( coyoteRequest.remoteAddr() );
  +             remoteHostMB.duplicate( coyoteRequest.remoteHost() );
  +         } catch(IOException iex) { // can't happen
  +         }
        }
        return remoteAddrMB;
       }
   
       public MessageBytes remoteHost() {
  +     if( remoteHostMB.isNull() ) {
  +         try {
  +             remoteHostMB.duplicate(coyoteRequest.remoteHost());
  +         } catch(IOException iex) { // can't happen
  +         }
  +     }
        if( remoteAddrMB.isNull() ) {
            coyoteRequest.action( ActionCode.ACTION_REQ_HOST_ATTRIBUTE, coyoteRequest 
);
  +         try{
  +             remoteAddrMB.duplicate( coyoteRequest.remoteAddr() );
  +             remoteHostMB.duplicate( coyoteRequest.remoteHost() );
  +         } catch(IOException iex) { // can't happen
  +         }
        }
        return remoteHostMB;
       }
  
  
  

--
To unsubscribe, e-mail:   <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>