tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From cos...@apache.org
Subject cvs commit: jakarta-tomcat-jasper/jasper34/runtime/org/apache/jasper34/runtime HttpJspBase.java
Date Sat, 16 Jun 2001 20:48:49 GMT
costin      01/06/16 13:48:49

  Modified:    jasper34/runtime/org/apache/jasper34/runtime
                        HttpJspBase.java
  Log:
  Eliminate the class loader property from HttpJspBase.
  
  There are few big problems with it:
  - it is not used by any active code
  - it doesn't work if we generate servlets with jspc ( it would require
  some magic in the container or a JspServlet overhead )
  - it doesn't work anyway for jsp pages using "extend" directive
  - hard to follow ( I never understood why it's here )
  
  Revision  Changes    Path
  1.6       +0 -12     jakarta-tomcat-jasper/jasper34/runtime/org/apache/jasper34/runtime/HttpJspBase.java
  
  Index: HttpJspBase.java
  ===================================================================
  RCS file: /home/cvs/jakarta-tomcat-jasper/jasper34/runtime/org/apache/jasper34/runtime/HttpJspBase.java,v
  retrieving revision 1.5
  retrieving revision 1.6
  diff -u -r1.5 -r1.6
  --- HttpJspBase.java	2001/06/09 21:29:02	1.5
  +++ HttpJspBase.java	2001/06/16 20:48:49	1.6
  @@ -78,8 +78,6 @@
       extends HttpServlet 
       implements HttpJspPage 
   {
  -    private ClassLoader cl;
  -
       protected PageContext pageContext;
       protected JspFactory _jspxFactory = null;
   
  @@ -100,16 +98,6 @@
   
       public final void destroy() {
   	jspDestroy();
  -    }
  -
  -    public final void setClassLoader(ClassLoader cl) {
  -        this.cl = cl;
  -    }
  -    
  -    protected ClassLoader getClassLoader() {
  -	if (cl == null) 
  -            return this.getClass().getClassLoader();
  -	return cl;
       }
   
       /**
  
  
  

Mime
View raw message