You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by kk...@apache.org on 2013/11/28 07:09:11 UTC

svn commit: r1546295 - /tomcat/trunk/java/org/apache/jasper/servlet/TldScanner.java

Author: kkolinko
Date: Thu Nov 28 06:09:11 2013
New Revision: 1546295

URL: http://svn.apache.org/r1546295
Log:
Review of r1546165
Do not waste time converting the whole path to lowercase. Convert only the filename, as it is its extension that we are testing here.

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=1546295&r1=1546294&r2=1546295&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/jasper/servlet/TldScanner.java (original)
+++ tomcat/trunk/java/org/apache/jasper/servlet/TldScanner.java Thu Nov 28 06:09:11 2013
@@ -319,7 +319,8 @@ public class TldScanner {
                 public FileVisitResult visitFile(Path file,
                                                  BasicFileAttributes attrs)
                         throws IOException {
-                    if (!file.toString().toLowerCase(Locale.ENGLISH).endsWith(TLD_EXT)) {
+                    if (!file.getFileName().toString()
+                            .toLowerCase(Locale.ENGLISH).endsWith(TLD_EXT)) {
                         return FileVisitResult.CONTINUE;
                     }
 



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