luehe 2004/09/08 13:02:12
Modified: jasper2/src/share/org/apache/jasper/servlet JspServlet.java
Log:
Simplified check for existence of JSP by replacing call to
context.getResourceAsStream() with context.getResource(), as suggested by Tim Funk
Revision Changes Path
1.38 +1 -7
jakarta-tomcat-jasper/jasper2/src/share/org/apache/jasper/servlet/JspServlet.java
Index: JspServlet.java
===================================================================
RCS file:
/home/cvs/jakarta-tomcat-jasper/jasper2/src/share/org/apache/jasper/servlet/JspServlet.java,v
retrieving revision 1.37
retrieving revision 1.38
diff -u -r1.37 -r1.38
--- JspServlet.java 21 Jul 2004 18:54:18 -0000 1.37
+++ JspServlet.java 8 Sep 2004 20:02:12 -0000 1.38
@@ -280,16 +280,10 @@
if (wrapper == null) {
// Check if the requested JSP page exists, to avoid
// creating unnecessary directories and files.
- InputStream resourceStream =
- context.getResourceAsStream(jspUri);
- if (resourceStream == null) {
+ if (null == context.getResource(jspUri)) {
response.sendError(HttpServletResponse.SC_NOT_FOUND,
jspUri);
return;
- } else {
- try {
- resourceStream.close();
- } catch(IOException e) { /* ignore */ }
}
boolean isErrorPage = exception != null;
wrapper = new JspServletWrapper(config, options, jspUri,
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
