You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by oh...@apache.org on 2013/07/13 21:29:35 UTC

svn commit: r1502849 - /commons/proper/configuration/trunk/src/main/java/org/apache/commons/configuration/ConfigurationUtils.java

Author: oheger
Date: Sat Jul 13 19:29:34 2013
New Revision: 1502849

URL: http://svn.apache.org/r1502849
Log:
Simplified ConfigurationUtils.convertToHierarchical().

There is no longer the need to disable list delimiter handling temporarily
because it is disabled for new configurations per default.

Modified:
    commons/proper/configuration/trunk/src/main/java/org/apache/commons/configuration/ConfigurationUtils.java

Modified: commons/proper/configuration/trunk/src/main/java/org/apache/commons/configuration/ConfigurationUtils.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/trunk/src/main/java/org/apache/commons/configuration/ConfigurationUtils.java?rev=1502849&r1=1502848&r2=1502849&view=diff
==============================================================================
--- commons/proper/configuration/trunk/src/main/java/org/apache/commons/configuration/ConfigurationUtils.java (original)
+++ commons/proper/configuration/trunk/src/main/java/org/apache/commons/configuration/ConfigurationUtils.java Sat Jul 13 19:29:34 2013
@@ -288,11 +288,9 @@ public final class ConfigurationUtils
                 hc.setExpressionEngine(engine);
             }
 
-            // Workaround for problem with copy()
-            boolean delimiterParsingStatus = hc.isDelimiterParsingDisabled();
-            hc.setDelimiterParsingDisabled(true);
+            // Per default, a DisabledListDelimiterHandler is set.
+            // So list delimiters in property values are not an issue.
             hc.append(conf);
-            hc.setDelimiterParsingDisabled(delimiterParsingStatus);
             return hc;
         }
     }