luehe 2002/12/03 15:17:48 Modified: jasper2/src/share/org/apache/jasper/compiler Generator.java jasper2/src/share/org/apache/jasper/runtime JspContextWrapper.java Log: Performance improvement: Pass ArrayList (instead of Vector) of scripting variables to JSP Context Wrapper constructor: ArrayList is not synchronized. Revision Changes Path 1.134 +12 -12 jakarta-tomcat-jasper/jasper2/src/share/org/apache/jasper/compiler/Generator.java Index: Generator.java =================================================================== RCS file: /home/cvs/jakarta-tomcat-jasper/jasper2/src/share/org/apache/jasper/compiler/Generator.java,v retrieving revision 1.133 retrieving revision 1.134 diff -u -r1.133 -r1.134 --- Generator.java 28 Nov 2002 04:18:08 -0000 1.133 +++ Generator.java 3 Dec 2002 23:17:48 -0000 1.134 @@ -3043,9 +3043,9 @@ out.pushIndent(); out.printil("super.setJspContext(ctx);"); TagVariableInfo[] tagVars = tagInfo.getTagVariableInfos(); - out.printil("java.util.Vector _jspx_nested = null;"); - out.printil("java.util.Vector _jspx_at_begin = null;"); - out.printil("java.util.Vector _jspx_at_end = null;"); + out.printil("java.util.ArrayList _jspx_nested = null;"); + out.printil("java.util.ArrayList _jspx_at_begin = null;"); + out.printil("java.util.ArrayList _jspx_at_end = null;"); for (int i=0; i<tagVars.length; i++) { @@ -3053,25 +3053,25 @@ case VariableInfo.NESTED: out.printil("if (_jspx_nested == null)"); out.pushIndent(); - out.printil("_jspx_nested = new java.util.Vector();"); + out.printil("_jspx_nested = new java.util.ArrayList();"); out.popIndent(); - out.printin("_jspx_nested.addElement("); + out.printin("_jspx_nested.add("); break; case VariableInfo.AT_BEGIN: out.printil("if (_jspx_at_begin == null)"); out.pushIndent(); - out.printil("_jspx_at_begin = new java.util.Vector();"); + out.printil("_jspx_at_begin = new java.util.ArrayList();"); out.popIndent(); - out.printin("_jspx_at_begin.addElement("); + out.printin("_jspx_at_begin.add("); break; case VariableInfo.AT_END: out.printil("if (_jspx_at_end == null)"); out.pushIndent(); - out.printil("_jspx_at_end = new java.util.Vector();"); + out.printil("_jspx_at_end = new java.util.ArrayList();"); out.popIndent(); - out.printin("_jspx_at_end.addElement("); + out.printin("_jspx_at_end.add("); break; } // switch 1.9 +12 -12 jakarta-tomcat-jasper/jasper2/src/share/org/apache/jasper/runtime/JspContextWrapper.java Index: JspContextWrapper.java =================================================================== RCS file: /home/cvs/jakarta-tomcat-jasper/jasper2/src/share/org/apache/jasper/runtime/JspContextWrapper.java,v retrieving revision 1.8 retrieving revision 1.9 diff -u -r1.8 -r1.9 --- JspContextWrapper.java 13 Nov 2002 17:40:41 -0000 1.8 +++ JspContextWrapper.java 3 Dec 2002 23:17:48 -0000 1.9 @@ -66,7 +66,7 @@ import java.util.Enumeration; import java.util.Hashtable; -import java.util.Vector; +import java.util.ArrayList; import java.util.Iterator; import javax.servlet.Servlet; @@ -106,19 +106,19 @@ private transient Hashtable pageAttributes; - // Vector of NESTED scripting variables - private Vector nestedVars; + // ArrayList of NESTED scripting variables + private ArrayList nestedVars; - // Vector of AT_BEGIN scripting variables - private Vector atBeginVars; + // ArrayList of AT_BEGIN scripting variables + private ArrayList atBeginVars; - // Vector of AT_END scripting variables - private Vector atEndVars; + // ArrayList of AT_END scripting variables + private ArrayList atEndVars; private Hashtable originalNestedVars; - public JspContextWrapper(JspContext jspContext, Vector nestedVars, - Vector atBeginVars, Vector atEndVars) { + public JspContextWrapper(JspContext jspContext, ArrayList nestedVars, + ArrayList atBeginVars, ArrayList atEndVars) { this.invokingJspCtxt = (PageContext) jspContext; this.nestedVars = nestedVars; this.atBeginVars = atBeginVars;
-- To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]> For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>