You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by re...@apache.org on 2002/05/30 02:02:35 UTC

cvs commit: jakarta-tomcat-jasper/jasper2/src/share/org/apache/jasper Constants.java

remm        02/05/29 17:02:35

  Modified:    jasper2/src/share/org/apache/jasper Constants.java
  Log:
  - The suggested amount was actually 512.
  - Patch submitted by Tyler Van Gorder <tkv at landacorp.com>. Sorry for forgetting
    to give you credit the first time around.
  
  Revision  Changes    Path
  1.3       +1 -1      jakarta-tomcat-jasper/jasper2/src/share/org/apache/jasper/Constants.java
  
  Index: Constants.java
  ===================================================================
  RCS file: /home/cvs/jakarta-tomcat-jasper/jasper2/src/share/org/apache/jasper/Constants.java,v
  retrieving revision 1.2
  retrieving revision 1.3
  diff -u -r1.2 -r1.3
  --- Constants.java	29 May 2002 23:24:59 -0000	1.2
  +++ Constants.java	30 May 2002 00:02:35 -0000	1.3
  @@ -134,7 +134,7 @@
       /**
        * Default size for the tag buffers.
        */
  -    public static final int DEFAULT_TAG_BUFFER_SIZE = 256;
  +    public static final int DEFAULT_TAG_BUFFER_SIZE = 512;
   
       /**
        * The query parameter that causes the JSP engine to just
  
  
  

--
To unsubscribe, e-mail:   <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>


Re: cvs commit: jakarta-tomcat-jasper/jasper2/src/share/org/apache/jasper Constants.java

Posted by peter lin <pe...@labs.gte.com>.
I'll give it a run through later today when I get home. too many things
to do at work right now. thanks for the info.

peter

Remy Maucherat wrote:
> 
> > hey remy,
> >
> > is this patch included in 4.1.3?
> 
> No.
> 
> Reusing the buffers and tags is the more interesting optimization. However,
> it still seemed useless to allocate big buffers, so I applied the patch.
> 
> > If so I'll test it out with the JSTL
> > pages I have.
> 
> Remy
> 
> --
> To unsubscribe, e-mail:   <ma...@jakarta.apache.org>
> For additional commands, e-mail: <ma...@jakarta.apache.org>

--
To unsubscribe, e-mail:   <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>


Re: cvs commit: jakarta-tomcat-jasper/jasper2/src/share/org/apache/jasper Constants.java

Posted by Remy Maucherat <re...@apache.org>.
> hey remy,
>
> is this patch included in 4.1.3?

No.

Reusing the buffers and tags is the more interesting optimization. However,
it still seemed useless to allocate big buffers, so I applied the patch.

> If so I'll test it out with the JSTL
> pages I have.

Remy


--
To unsubscribe, e-mail:   <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>


Re: cvs commit: jakarta-tomcat-jasper/jasper2/src/share/org/apache/jasper Constants.java

Posted by peter lin <pe...@labs.gte.com>.
hey remy,

is this patch included in 4.1.3?  If so I'll test it out with the JSTL
pages I have.

peter


remm@apache.org wrote:
> 
> remm        02/05/29 17:02:35
> 
>   Modified:    jasper2/src/share/org/apache/jasper Constants.java
>   Log:
>   - The suggested amount was actually 512.
>   - Patch submitted by Tyler Van Gorder <tkv at landacorp.com>. Sorry for forgetting
>     to give you credit the first time around.
> 
>   Revision  Changes    Path
>   1.3       +1 -1      jakarta-tomcat-jasper/jasper2/src/share/org/apache/jasper/Constants.java
> 
>   Index: Constants.java
>   ===================================================================
>   RCS file: /home/cvs/jakarta-tomcat-jasper/jasper2/src/share/org/apache/jasper/Constants.java,v
>   retrieving revision 1.2
>   retrieving revision 1.3
>   diff -u -r1.2 -r1.3
>   --- Constants.java    29 May 2002 23:24:59 -0000      1.2
>   +++ Constants.java    30 May 2002 00:02:35 -0000      1.3
>   @@ -134,7 +134,7 @@
>        /**
>         * Default size for the tag buffers.
>         */
>   -    public static final int DEFAULT_TAG_BUFFER_SIZE = 256;
>   +    public static final int DEFAULT_TAG_BUFFER_SIZE = 512;
> 
>        /**
>         * The query parameter that causes the JSP engine to just
> 
> 
> 
> 
> --
> To unsubscribe, e-mail:   <ma...@jakarta.apache.org>
> For additional commands, e-mail: <ma...@jakarta.apache.org>

--
To unsubscribe, e-mail:   <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>