You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by he...@apache.org on 2012/03/13 11:45:37 UTC

svn commit: r1300063 - /commons/proper/jexl/trunk/src/main/java/org/apache/commons/jexl3/internal/TemplateEngine.java

Author: henrib
Date: Tue Mar 13 10:45:36 2012
New Revision: 1300063

URL: http://svn.apache.org/viewvc?rev=1300063&view=rev
Log:
Removed unecessary initializations

Modified:
    commons/proper/jexl/trunk/src/main/java/org/apache/commons/jexl3/internal/TemplateEngine.java

Modified: commons/proper/jexl/trunk/src/main/java/org/apache/commons/jexl3/internal/TemplateEngine.java
URL: http://svn.apache.org/viewvc/commons/proper/jexl/trunk/src/main/java/org/apache/commons/jexl3/internal/TemplateEngine.java?rev=1300063&r1=1300062&r2=1300063&view=diff
==============================================================================
--- commons/proper/jexl/trunk/src/main/java/org/apache/commons/jexl3/internal/TemplateEngine.java (original)
+++ commons/proper/jexl/trunk/src/main/java/org/apache/commons/jexl3/internal/TemplateEngine.java Tue Mar 13 10:45:36 2012
@@ -70,7 +70,7 @@ public final class TemplateEngine extend
     public TemplateEngine(Engine aJexl, int cacheSize) {
         this(aJexl, cacheSize, '$', '#');
     }
-    
+
     /**
      * Creates a new instance of {@link JxltEngine} creating a local cache.
      * @param aJexl the JexlEngine to use.
@@ -586,7 +586,7 @@ public final class TemplateEngine extend
         @Override
         protected Object evaluate(Interpreter interpreter) {
             final int size = exprs.length;
-            Object value = null;
+            Object value;
             // common case: evaluate all expressions & concatenate them as a string
             StringBuilder strb = new StringBuilder();
             for (int e = 0; e < size; ++e) {
@@ -777,7 +777,7 @@ public final class TemplateEngine extend
                             inner -= 1;
                         } else {
                             // materialize the nested/deferred expr
-                            TemplateExpression dexpr = null;
+                            TemplateExpression dexpr;
                             if (nested) {
                                 dexpr = new NestedExpression(
                                         expr.substring(inested, i + 1),
@@ -843,7 +843,7 @@ public final class TemplateEngine extend
         private final String body;
 
         /**
-         * Creates a new block. 
+         * Creates a new block.
          * @param theType the type
          * @param theBlock the content
          */
@@ -1057,8 +1057,8 @@ public final class TemplateEngine extend
         public Object resolveNamespace(String ns) {
             if ("jexl".equals(ns)) {
                 return this;
-            } else if (wrap instanceof NamespaceResolver) {
-                return ((NamespaceResolver) wrap).resolveNamespace(ns);
+            } else if (wrap instanceof JexlContext.NamespaceResolver) {
+                return ((JexlContext.NamespaceResolver) wrap).resolveNamespace(ns);
             } else {
                 return null;
             }
@@ -1100,7 +1100,7 @@ public final class TemplateEngine extend
         protected void printComposite(CompositeExpression composite) {
             TemplateExpression[] cexprs = composite.exprs;
             final int size = cexprs.length;
-            Object value = null;
+            Object value;
             for (int e = 0; e < size; ++e) {
                 value = cexprs[e].evaluate(this);
                 doPrint(value);
@@ -1165,7 +1165,7 @@ public final class TemplateEngine extend
      */
     protected List<Block> readTemplate(final String prefix, Reader source) {
         try {
-            int prefixLen = prefix.length();
+            int prefixLen;
             List<Block> blocks = new ArrayList<Block>();
             BufferedReader reader;
             if (source instanceof BufferedReader) {