You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jackrabbit.apache.org by db...@apache.org on 2011/10/08 05:56:05 UTC
svn commit: r1180307 -
/jackrabbit/trunk/jackrabbit-jcr-commons/src/main/java/org/apache/jackrabbit/commons/JcrUtils.java
Author: dbrosius
Date: Sat Oct 8 03:56:05 2011
New Revision: 1180307
URL: http://svn.apache.org/viewvc?rev=1180307&view=rev
Log:
don't allocate map unless it's needed
Modified:
jackrabbit/trunk/jackrabbit-jcr-commons/src/main/java/org/apache/jackrabbit/commons/JcrUtils.java
Modified: jackrabbit/trunk/jackrabbit-jcr-commons/src/main/java/org/apache/jackrabbit/commons/JcrUtils.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-jcr-commons/src/main/java/org/apache/jackrabbit/commons/JcrUtils.java?rev=1180307&r1=1180306&r2=1180307&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-jcr-commons/src/main/java/org/apache/jackrabbit/commons/JcrUtils.java (original)
+++ jackrabbit/trunk/jackrabbit-jcr-commons/src/main/java/org/apache/jackrabbit/commons/JcrUtils.java Sat Oct 8 03:56:05 2011
@@ -140,12 +140,12 @@ public class JcrUtils {
if (parameters != null
&& parameters.containsKey(JcrUtils.REPOSITORY_URI)) {
String uri = parameters.get(JcrUtils.REPOSITORY_URI);
- Map<String, String> copy = new HashMap<String, String>(parameters);
try {
URI u = new URI(uri);
String query = u.getRawQuery();
if (query != null) {
- for (String entry : query.split("&")) {
+ Map<String, String> copy = new HashMap<String, String>(parameters);
+ for (String entry : query.split("&")) {
int i = entry.indexOf('=');
if (i != -1) {
copy.put(