On 09/02/2015 08:57 AM, cbris...@apache.org wrote: > Author: cbrisson > Date: Wed Sep 2 12:57:21 2015 > New Revision: 1700798 > > URL: http://svn.apache.org/r1700798 > Log: > [tools] switch to SLF4J logging facade >
[...] > Modified: > velocity/tools/trunk/velocity-tools-generic/src/main/java/org/apache/velocity/tools/generic/ValueParser.java > URL: > http://svn.apache.org/viewvc/velocity/tools/trunk/velocity-tools-generic/src/main/java/org/apache/velocity/tools/generic/ValueParser.java?rev=1700798&r1=1700797&r2=1700798&view=diff > ============================================================================== > --- > velocity/tools/trunk/velocity-tools-generic/src/main/java/org/apache/velocity/tools/generic/ValueParser.java > (original) > +++ > velocity/tools/trunk/velocity-tools-generic/src/main/java/org/apache/velocity/tools/generic/ValueParser.java > Wed Sep 2 12:57:21 2015 > @@ -22,7 +22,7 @@ package org.apache.velocity.tools.generi > import java.util.Map; > import java.util.Locale; > import java.util.Set; > -import java.util.HashMap; > +import java.util.TreeMap; > import java.util.Collection; > > import org.apache.velocity.tools.config.DefaultKey; > @@ -68,7 +68,10 @@ public class ValueParser extends Convers > */ > public static final String READONLY_KEY = "readOnly"; > > - public ValueParser() {} > + public ValueParser() > + { > + source = new TreeMap<String,Object>(); > + } > > public ValueParser(Map<String,Object> source) > { > @@ -528,11 +531,6 @@ public class ValueParser extends Convers > if (entry.getKey().startsWith(subkey) && > entry.getKey().length() > subkey.length()) > { > - if(values == null) > - { > - values = new HashMap<String,Object>(); > - } > - This removal looks like a bug, the "values" variable is never initialized. It also doesn't have anything to do with the logger switch, so it shouldn't have been included in this commit. > > values.put(entry.getKey().substring(subkey.length()),entry.getValue()); > } > } > -- Sergiu Dumitriu http://purl.org/net/sergiu/ --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@velocity.apache.org For additional commands, e-mail: dev-h...@velocity.apache.org