You are viewing a plain text version of this content. The canonical link for it is here.
Posted to oak-commits@jackrabbit.apache.org by ch...@apache.org on 2016/06/03 04:29:18 UTC
svn commit: r1746665 -
/jackrabbit/oak/trunk/oak-commons/src/main/java/org/apache/jackrabbit/oak/commons/PathUtils.java
Author: chetanm
Date: Fri Jun 3 04:29:18 2016
New Revision: 1746665
URL: http://svn.apache.org/viewvc?rev=1746665&view=rev
Log:
OAK-4424 - Optimize PathUtils.concat by using a properly sized StringBuilder
Modified:
jackrabbit/oak/trunk/oak-commons/src/main/java/org/apache/jackrabbit/oak/commons/PathUtils.java
Modified: jackrabbit/oak/trunk/oak-commons/src/main/java/org/apache/jackrabbit/oak/commons/PathUtils.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-commons/src/main/java/org/apache/jackrabbit/oak/commons/PathUtils.java?rev=1746665&r1=1746664&r2=1746665&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-commons/src/main/java/org/apache/jackrabbit/oak/commons/PathUtils.java (original)
+++ jackrabbit/oak/trunk/oak-commons/src/main/java/org/apache/jackrabbit/oak/commons/PathUtils.java Fri Jun 3 04:29:18 2016
@@ -315,7 +315,8 @@ public final class PathUtils {
} else if (isAbsolutePath(subPath)) {
throw new IllegalArgumentException("Cannot append absolute path " + subPath);
}
- StringBuilder buff = new StringBuilder(parentPath);
+ StringBuilder buff = new StringBuilder(parentPath.length() + subPath.length() + 1);
+ buff.append(parentPath);
if (!denotesRootPath(parentPath)) {
buff.append('/');
}