You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by ol...@apache.org on 2012/07/30 15:55:47 UTC
svn commit: r1367093 -
/maven/indexer/trunk/indexer-core/src/main/java/org/apache/maven/index/treeview/DefaultIndexTreeView.java
Author: olamy
Date: Mon Jul 30 13:55:47 2012
New Revision: 1367093
URL: http://svn.apache.org/viewvc?rev=1367093&view=rev
Log:
indexOf (char) is faster than indexOf(String)
Modified:
maven/indexer/trunk/indexer-core/src/main/java/org/apache/maven/index/treeview/DefaultIndexTreeView.java
Modified: maven/indexer/trunk/indexer-core/src/main/java/org/apache/maven/index/treeview/DefaultIndexTreeView.java
URL: http://svn.apache.org/viewvc/maven/indexer/trunk/indexer-core/src/main/java/org/apache/maven/index/treeview/DefaultIndexTreeView.java?rev=1367093&r1=1367092&r2=1367093&view=diff
==============================================================================
--- maven/indexer/trunk/indexer-core/src/main/java/org/apache/maven/index/treeview/DefaultIndexTreeView.java (original)
+++ maven/indexer/trunk/indexer-core/src/main/java/org/apache/maven/index/treeview/DefaultIndexTreeView.java Mon Jul 30 13:55:47 2012
@@ -389,14 +389,14 @@ public class DefaultIndexTreeView
// 2nd try, lets consider path a group + artifactId, we must ensure there is at least one / but not as root
- if ( path.lastIndexOf( "/" ) > 0 )
+ if ( path.lastIndexOf( '/' ) > 0 )
{
// reset wp
wp = path;
- a = wp.substring( wp.lastIndexOf( "/" ) + 1, wp.length() );
+ a = wp.substring( wp.lastIndexOf( '/' ) + 1, wp.length() );
- g = wp.substring( 1, wp.lastIndexOf( "/" ) ).replace( '/', '.' );
+ g = wp.substring( 1, wp.lastIndexOf( '/' ) ).replace( '/', '.' );
result = getArtifactsByGA( g, a, request );
@@ -416,13 +416,13 @@ public class DefaultIndexTreeView
// reset wp
wp = path;
- v = wp.substring( wp.lastIndexOf( "/" ) + 1, wp.length() );
+ v = wp.substring( wp.lastIndexOf( '/' ) + 1, wp.length() );
- wp = wp.substring( 0, wp.lastIndexOf( "/" ) );
+ wp = wp.substring( 0, wp.lastIndexOf( '/' ) );
- a = wp.substring( wp.lastIndexOf( "/" ) + 1, wp.length() );
+ a = wp.substring( wp.lastIndexOf( '/' ) + 1, wp.length() );
- g = wp.substring( 1, wp.lastIndexOf( "/" ) ).replace( '/', '.' );
+ g = wp.substring( 1, wp.lastIndexOf( '/' ) ).replace( '/', '.' );
result = getArtifactsByGAV( g, a, v, request );