You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by si...@apache.org on 2011/05/31 07:28:00 UTC

svn commit: r1129491 - /commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/substitution/VariableSubstitutor.java

Author: simonetripodi
Date: Tue May 31 05:27:59 2011
New Revision: 1129491

URL: http://svn.apache.org/viewvc?rev=1129491&view=rev
Log:
VariableSubstitutor members can be final

Modified:
    commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/substitution/VariableSubstitutor.java

Modified: commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/substitution/VariableSubstitutor.java
URL: http://svn.apache.org/viewvc/commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/substitution/VariableSubstitutor.java?rev=1129491&r1=1129490&r2=1129491&view=diff
==============================================================================
--- commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/substitution/VariableSubstitutor.java (original)
+++ commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/substitution/VariableSubstitutor.java Tue May 31 05:27:59 2011
@@ -37,18 +37,18 @@ public class VariableSubstitutor
     /**
      * The expander to be used to expand variables in the attributes. Null when no expansion should be performed.
      */
-    private VariableExpander attributesExpander;
+    private final VariableExpander attributesExpander;
 
     /**
      * Attributes implementation that (lazily) performs variable substitution. Will be lazily created when needed then
      * reused.
      */
-    private VariableAttributes variableAttributes;
+    private final VariableAttributes variableAttributes;
 
     /**
      * The expander to be used to expand variables in the body text. Null when no expansion should be performed.
      */
-    private VariableExpander bodyTextExpander;
+    private final VariableExpander bodyTextExpander;
 
     /**
      * Constructs a Substitutor which uses the same VariableExpander for both body text and attibutes.