You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jmeter.apache.org by pm...@apache.org on 2012/09/05 22:31:04 UTC
svn commit: r1381345 -
/jmeter/trunk/src/jorphan/org/apache/jorphan/reflect/ClassFinder.java
Author: pmouawad
Date: Wed Sep 5 20:31:04 2012
New Revision: 1381345
URL: http://svn.apache.org/viewvc?rev=1381345&view=rev
Log:
use indexOf(char)
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=1381345&r1=1381344&r2=1381345&view=diff
==============================================================================
--- jmeter/trunk/src/jorphan/org/apache/jorphan/reflect/ClassFinder.java (original)
+++ jmeter/trunk/src/jorphan/org/apache/jorphan/reflect/ClassFinder.java Wed Sep 5 20:31:04 2012
@@ -102,7 +102,7 @@ public final class ClassFinder {
if (notContains!=null && s.indexOf(notContains) != -1){
return false; // It contains a banned string
}
- if ((s.indexOf("$") == -1) || inner) { // $NON-NLS-1$
+ if ((s.indexOf('$') == -1) || inner) { // $NON-NLS-1$
if (isChildOf(parents,s, contextClassLoader)) {
return super.add(s);
}
@@ -138,7 +138,7 @@ public final class ClassFinder {
if (contains(s)) {
return false;// No need to check it again
}
- if ((s.indexOf("$") == -1) || inner) { // $NON-NLS-1$
+ if ((s.indexOf('$') == -1) || inner) { // $NON-NLS-1$
if (hasAnnotationOnMethod(annotations,s, contextClassLoader)) {
return super.add(s);
}
@@ -558,7 +558,7 @@ public final class ClassFinder {
} else if (list[i].endsWith(DOT_CLASS) && file.exists() && (file.length() != 0)) {
final String path = file.getPath();
listClasses.add(path.substring(strPathElement.length() + 1,
- path.lastIndexOf(".")) // $NON-NLS-1$
+ path.lastIndexOf('.')) // $NON-NLS-1$
.replace(File.separator.charAt(0), '.')); // $NON-NLS-1$
}
}