svn commit: r959435 - /maven/shared/trunk/maven-doxia-tools/src/main/java/org/apache/maven/doxia/tools/DefaultSiteTool.java

[hboutemy]

Author: hboutemy
Date: Wed Jun 30 21:13:06 2010
New Revision: 959435

URL: http://svn.apache.org/viewvc?rev=959435&view=rev
Log:
use StringBuilder instead of String +=

Modified:
    
maven/shared/trunk/maven-doxia-tools/src/main/java/org/apache/maven/doxia/tools/DefaultSiteTool.java

Modified: 
maven/shared/trunk/maven-doxia-tools/src/main/java/org/apache/maven/doxia/tools/DefaultSiteTool.java
URL: 
http://svn.apache.org/viewvc/maven/shared/trunk/maven-doxia-tools/src/main/java/org/apache/maven/doxia/tools/DefaultSiteTool.java?rev=959435&r1=959434&r2=959435&view=diff
==============================================================================
--- 
maven/shared/trunk/maven-doxia-tools/src/main/java/org/apache/maven/doxia/tools/DefaultSiteTool.java
 (original)
+++ 
maven/shared/trunk/maven-doxia-tools/src/main/java/org/apache/maven/doxia/tools/DefaultSiteTool.java
 Wed Jun 30 21:13:06 2010
@@ -356,43 +356,43 @@ public class DefaultSiteTool
             toTokeniser.nextToken();
         }
 
-        String relativePath = "";
+        StringBuilder relativePath = new StringBuilder();
 
         // add back refs for the rest of from location.
         while ( fromTokeniser.hasMoreTokens() )
         {
             fromTokeniser.nextToken();
 
-            relativePath += "..";
+            relativePath.append( ".." );
 
             if ( fromTokeniser.hasMoreTokens() )
             {
-                relativePath += File.separatorChar;
+                relativePath.append( File.separatorChar );
             }
         }
 
         if ( relativePath.length() != 0 && toTokeniser.hasMoreTokens() )
         {
-            relativePath += File.separatorChar;
+            relativePath.append( File.separatorChar );
         }
 
         // add fwd fills for whatever's left of to.
         while ( toTokeniser.hasMoreTokens() )
         {
-            relativePath += toTokeniser.nextToken();
+            relativePath.append( toTokeniser.nextToken() );
 
             if ( toTokeniser.hasMoreTokens() )
             {
-                relativePath += File.separatorChar;
+                relativePath.append( File.separatorChar );
             }
         }
 
-        if ( !relativePath.equals( to ) )
+        if ( getLogger().isDebugEnabled() && !relativePath.equals( to ) )
         {
             getLogger().debug( "Mapped url: " + to + " to relative path: " + 
relativePath );
         }
 
-        return relativePath;
+        return relativePath.toString();
     }
 
     /** {...@inheritdoc} */