FAQ
Author: markt
Date: Thu Oct 30 10:27:44 2014
New Revision: 1635456

URL: http://svn.apache.org/r1635456
Log:
Fix bug highlighted by test case failure after r1635154. If the webappPath is null, the resourcePath should be null.
Also fix some Java8 Javadoc issues.

Modified:
     tomcat/trunk/java/org/apache/jasper/servlet/TldScanner.java

Modified: tomcat/trunk/java/org/apache/jasper/servlet/TldScanner.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/jasper/servlet/TldScanner.java?rev=1635456&r1=1635455&r2=1635456&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/jasper/servlet/TldScanner.java (original)
+++ tomcat/trunk/java/org/apache/jasper/servlet/TldScanner.java Thu Oct 30 10:27:44 2014
@@ -66,9 +66,16 @@ public class TldScanner {
      private final List<String> listeners = new ArrayList<>();

      /**
- * Initialize with the application's ServletContext.
+ * Initialise with the application's ServletContext.
       *
- * @param context the application's servletContext
+ * @param context the application's servletContext
+ * @param namespaceAware should the XML parser used to parse TLD files be
+ * configured to be name space aware
+ * @param validation should the XML parser used to parse TLD files be
+ * configured to use validation
+ * @param blockExternal should the XML parser used to parse TLD files be
+ * configured to be block references to external
+ * entities
       */
      public TldScanner(ServletContext context,
                        boolean namespaceAware,
@@ -129,7 +136,10 @@ public class TldScanner {

      /**
       * Set the class loader used by the digester to create objects as a result
- * of this scan. Normally this only needs tobe set when using JspC.
+ * of this scan. Normally this only needs to be set when using JspC.
+ *
+ * @param classLoader Class loader to use when creating new objects while
+ * parsing TLDs
       */
      public void setClassLoader(ClassLoader classLoader) {
          tldParser.setClassLoader(classLoader);
@@ -330,12 +340,17 @@ public class TldScanner {
                          return FileVisitResult.CONTINUE;
                      }

- String subPath = file.subpath(
- filePath.getNameCount(), file.getNameCount()).toString();
- if ('/' != File.separatorChar) {
- subPath = subPath.replace(File.separatorChar, '/');
+ String resourcePath;
+ if (webappPath == null) {
+ resourcePath = null;
+ } else {
+ String subPath = file.subpath(
+ filePath.getNameCount(), file.getNameCount()).toString();
+ if ('/' != File.separatorChar) {
+ subPath = subPath.replace(File.separatorChar, '/');
+ }
+ resourcePath = webappPath + "/" + subPath;
                      }
- String resourcePath = webappPath + "/" + subPath;

                      try {
                          URL url = file.toUri().toURL();



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org

Search Discussions

Related Discussions

Discussion Navigation
viewthread | post
posts ‹ prev | 1 of 1 | next ›
Discussion Overview
groupdev @
categoriestomcat
postedOct 30, '14 at 10:29a
activeOct 30, '14 at 10:29a
posts1
users1
websitetomcat.apache.org
irc#tomcat

1 user in discussion

Markt: 1 post

People

Translate

site design / logo © 2017 Grokbase