Re: [PROPOSED 3.2B7 PATCH] error displayed for case mismatch in JSP U RL

[cm...@yahoo.com]

+1

Costin

> ===== JspCompiler.java Patch =====
> RCS file: /home/cvs/jakarta-tomcat/src/share/org/apache/jasper/compiler/JspCompiler.java,v
> retrieving revision 1.14.2.1
> diff -u -r1.14.2.1 JspCompiler.java
> --- JspCompiler.java	2000/07/25 20:45:09	1.14.2.1
> +++ JspCompiler.java	2000/11/15 12:13:12
> @@ -304,7 +304,11 @@
>      public boolean isOutDated() {
>          File jspReal = null;
>  
> -        jspReal = new File(ctxt.getRealPath(jsp.getPath()));
> +        String realPath = ctxt.getRealPath(jsp.getPath());
> +        if (realPath == null)
> +            return true;
> +
> +        jspReal = new File(realPath);
>  
>          File classFile = new File(getClassFileName());
>          if (classFile.exists()) {
> ==================================
> 
> ===== JspReader.java patch =====
> RCS file: /home/cvs/jakarta-tomcat/src/share/org/apache/jasper/compiler/JspReader.java,v
> retrieving revision 1.16.2.4
> diff -u -r1.16.2.4 JspReader.java
> --- JspReader.java	2000/11/06 04:08:14	1.16.2.4
> +++ JspReader.java	2000/11/15 12:12:29
> @@ -171,6 +171,9 @@
>  	    ? file.getAbsolutePath()
>  	    : context.getRealPath(file.toString());
>  
> +	if (longName == null)
> +	    throw new FileNotFoundException(file.toString());
> +
>  	int fileid = registerSourceFile(longName);
>  	
>          if (fileid == -1)
> ================================
> 
> This probably won't eliminate the head scratching, but at least they will
> be scratching their heads for the right reason. :-)  Also, this will make
> section 6.7 in the Readme correct about the error seen due to a case mismatch.
> 
> If no one objects, I'll commit these changes this afternoon.
> 
> Cheers,
> Larry
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: tomcat-dev-unsubscribe@jakarta.apache.org
> For additional commands, e-mail: tomcat-dev-help@jakarta.apache.org
>