You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@tomcat.apache.org by na...@locus.apache.org on 2000/11/06 16:07:42 UTC

cvs commit: jakarta-tomcat/src/facade22/org/apache/tomcat/facade ServletWrapper.java

nacho       00/11/06 07:07:41

  Modified:    src/facade22/org/apache/tomcat/facade ServletWrapper.java
  Log:
  A peformance tip took from tomcat-dev
  
  Thanks to Sam Ruby to point out this
  
  Revision  Changes    Path
  1.10      +3 -1      jakarta-tomcat/src/facade22/org/apache/tomcat/facade/ServletWrapper.java
  
  Index: ServletWrapper.java
  ===================================================================
  RCS file: /home/cvs/jakarta-tomcat/src/facade22/org/apache/tomcat/facade/ServletWrapper.java,v
  retrieving revision 1.9
  retrieving revision 1.10
  diff -u -r1.9 -r1.10
  --- ServletWrapper.java	2000/11/02 21:51:40	1.9
  +++ ServletWrapper.java	2000/11/06 15:07:40	1.10
  @@ -351,9 +351,11 @@
       {
   	// The servlet is loaded and not null - otherwise init()
   	// throws exception
  +        if( initialized )
  +            return;
   	try {
   	    // if multiple threads will call the same servlet at once,
  -	    // we should have only one init 
  +	    // we should have only one init
   	    synchronized( this ) {
   		// we may have 2 threads entering doInit,
   		// the first one may init the servlet