You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jmeter.apache.org by se...@apache.org on 2017/02/23 15:03:42 UTC

svn commit: r1784148 - /jmeter/trunk/src/jorphan/org/apache/jorphan/reflect/ClassFinder.java

Author: sebb
Date: Thu Feb 23 15:03:42 2017
New Revision: 1784148

URL: http://svn.apache.org/viewvc?rev=1784148&view=rev
Log:
testFindAllClassesInJar(org.apache.jorphan.reflect.TestClassFinder) can fail on Windows
Bugzilla Id: 60771

Modified:
    jmeter/trunk/src/jorphan/org/apache/jorphan/reflect/ClassFinder.java

Modified: jmeter/trunk/src/jorphan/org/apache/jorphan/reflect/ClassFinder.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/jorphan/org/apache/jorphan/reflect/ClassFinder.java?rev=1784148&r1=1784147&r2=1784148&view=diff
==============================================================================
--- jmeter/trunk/src/jorphan/org/apache/jorphan/reflect/ClassFinder.java (original)
+++ jmeter/trunk/src/jorphan/org/apache/jorphan/reflect/ClassFinder.java Thu Feb 23 15:03:42 2017
@@ -30,6 +30,7 @@ import java.util.Collections;
 import java.util.Enumeration;
 import java.util.HashSet;
 import java.util.List;
+import java.util.Locale;
 import java.util.Set;
 import java.util.StringTokenizer;
 import java.util.TreeSet;
@@ -342,6 +343,7 @@ public final class ClassFinder {
     /**
      * Fix a path:
      * - replace "." by current directory
+     * - upcase the first character if it appears to be a drive letter
      * - trim any trailing spaces
      * - replace \ by /
      * - replace // by /
@@ -354,6 +356,9 @@ public final class ClassFinder {
         if (path.equals(".")) { // $NON-NLS-1$
             return System.getProperty("user.dir"); // $NON-NLS-1$
         }
+        if (path.length() > 3 && path.matches("[a-z]:\\.*")) { // lower-case drive letter?
+            path = path.substring(0, 1).toUpperCase(Locale.ROOT) + path.substring(1);
+        }
         path = path.trim().replace('\\', '/'); // $NON-NLS-1$ // $NON-NLS-2$
         path = JOrphanUtils.substitute(path, "//", "/"); // $NON-NLS-1$// $NON-NLS-2$