You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2009/07/10 18:28:41 UTC

svn commit: r793021 - /tomcat/trunk/java/org/apache/catalina/startup/TldConfig.java

Author: markt
Date: Fri Jul 10 16:28:41 2009
New Revision: 793021

URL: http://svn.apache.org/viewvc?rev=793021&view=rev
Log:
Better notes for future reference

Modified:
    tomcat/trunk/java/org/apache/catalina/startup/TldConfig.java

Modified: tomcat/trunk/java/org/apache/catalina/startup/TldConfig.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/startup/TldConfig.java?rev=793021&r1=793020&r2=793021&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/startup/TldConfig.java (original)
+++ tomcat/trunk/java/org/apache/catalina/startup/TldConfig.java Fri Jul 10 16:28:41 2009
@@ -508,8 +508,10 @@
         String taglibs[] = context.findTaglibs();
         for (int i = 0; i < taglibs.length; i++) {
             String resourcePath = context.findTaglib(taglibs[i]);
-            // FIXME - Servlet 2.4 DTD implies that the location MUST be
-            // a context-relative path starting with '/'?
+            // Note: Whilst the Servlet 2.4 DTD implies that the location must
+            // be a context-relative path starting with '/', JSP.7.3.6.1 states
+            // explicitly how paths that do not start with '/' should be
+            // handled.
             if (!resourcePath.startsWith("/")) {
                 resourcePath = "/WEB-INF/" + resourcePath;
             }



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