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(