yoavs       2005/03/23 08:38:05

  Modified:    webapps/docs changelog.xml
               jasper2/src/share/org/apache/jasper/servlet
                        JspServletWrapper.java
  Log:
  Bugzilla 33219.
  
  Revision  Changes    Path
  1.254     +3 -0      jakarta-tomcat-catalina/webapps/docs/changelog.xml
  
  Index: changelog.xml
  ===================================================================
  RCS file: /home/cvs/jakarta-tomcat-catalina/webapps/docs/changelog.xml,v
  retrieving revision 1.253
  retrieving revision 1.254
  diff -u -r1.253 -r1.254
  --- changelog.xml     23 Mar 2005 16:31:41 -0000      1.253
  +++ changelog.xml     23 Mar 2005 16:38:05 -0000      1.254
  @@ -278,6 +278,9 @@
         <fix>
           <bug>33539</bug>: Better error message when an unknown element is 
encountered in the tag file. (yoavs)
         </fix>
  +      <fix>
  +        <bug>33219</bug>: Minor JspServletWrapper code cleanup. (yoavs)
  +      </fix>
       </changelog>
     </subsection>
     
  
  
  
  1.40      +3 -6      
jakarta-tomcat-jasper/jasper2/src/share/org/apache/jasper/servlet/JspServletWrapper.java
  
  Index: JspServletWrapper.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-tomcat-jasper/jasper2/src/share/org/apache/jasper/servlet/JspServletWrapper.java,v
  retrieving revision 1.39
  retrieving revision 1.40
  diff -u -r1.39 -r1.40
  --- JspServletWrapper.java    21 Mar 2005 15:38:12 -0000      1.39
  +++ JspServletWrapper.java    23 Mar 2005 16:38:05 -0000      1.40
  @@ -196,9 +196,7 @@
               }
               if (options.getDevelopment() || firstTime ) {
                   synchronized (this) {
  -                    if (firstTime) {
  -                        firstTime = false;
  -                    }
  +                    firstTime = false;
                       ctxt.compile();
                   }
               } else {
  @@ -289,9 +287,8 @@
                */
               if (options.getDevelopment() || firstTime ) {
                   synchronized (this) {
  -                    if (firstTime) {
  -                        firstTime = false;
  -                    }
  +                    firstTime = false;
  +
                       // The following sets reload to true, if necessary
                       ctxt.compile();
                   }
  
  
  

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

Reply via email to