Author: markt
Date: Mon Nov 19 17:26:00 2018
New Revision: 1846928
URL: http://svn.apache.org/viewvc?rev=1846928&view=rev
Log:
The docBase is required and has been for ~15 years (r305684)
Modified:
tomcat/trunk/java/org/apache/jasper/JspC.java
Modified: tomcat/trunk/java/org/apache/jasper/JspC.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/jasper/JspC.java?rev=1846928&r1=1846927&r2=1846928&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/jasper/JspC.java (original)
+++ tomcat/trunk/java/org/apache/jasper/JspC.java Mon Nov 19 17:26:00 2018
@@ -1769,7 +1769,7 @@ public class JspC extends Task implement
/**
* Find the WEB-INF dir by looking up in the directory tree.
* This is used if no explicit docbase is set, but only files.
- * XXX Maybe we should require the docbase.
+ *
* @param f The path from which it will start looking
*/
protected void locateUriRoot( File f ) {
@@ -1804,8 +1804,7 @@ public class JspC extends Task implement
}
// If there is no acceptable candidate, uriRoot will
- // remain null to indicate to the CompilerContext to
- // use the current working/user dir.
+ // remain null.
}
if (uriRoot != null) {
@@ -1814,9 +1813,7 @@ public class JspC extends Task implement
}
}
} catch (IOException ioe) {
- // since this is an optional default and a null value
- // for uriRoot has a non-error meaning, we can just
- // pass straight through
+ // Missing uriRoot will be handled in the caller.
}
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
