Don't need to clear eval result if we are no longer using caching for eager execution of ast nodes

Author: jasmith-hs

src/main/java/com/hubspot/jinjava/el/ext/eager/EagerAstBinary.java

@@ -82,14 +82,6 @@ public void setEvalResult(Object evalResult) {
     hasEvalResult = true;
   }
 
-  @Override
-  public void clearEvalResult() {
-    evalResult = null;
-    hasEvalResult = false;
-    left.clearEvalResult();
-    right.clearEvalResult();
-  }
-
   @Override
   public boolean hasEvalResult() {
     return hasEvalResult;

src/main/java/com/hubspot/jinjava/el/ext/eager/EagerAstBracket.java

@@ -46,16 +46,6 @@ public void setEvalResult(Object evalResult) {
     hasEvalResult = true;
   }
 
-  @Override
-  public void clearEvalResult() {
-    evalResult = null;
-    hasEvalResult = false;
-    ((EvalResultHolder) prefix).clearEvalResult();
-    if (property != null) {
-      ((EvalResultHolder) property).clearEvalResult();
-    }
-  }
-
   @Override
   public boolean hasEvalResult() {
     return hasEvalResult;

src/main/java/com/hubspot/jinjava/el/ext/eager/EagerAstChoice.java

@@ -61,15 +61,6 @@ public void setEvalResult(Object evalResult) {
     hasEvalResult = true;
   }
 
-  @Override
-  public void clearEvalResult() {
-    evalResult = null;
-    hasEvalResult = false;
-    question.clearEvalResult();
-    yes.clearEvalResult();
-    no.clearEvalResult();
-  }
-
   @Override
   public boolean hasEvalResult() {
     return hasEvalResult;

src/main/java/com/hubspot/jinjava/el/ext/eager/EagerAstDict.java

@@ -96,23 +96,6 @@ public void setEvalResult(Object evalResult) {
     hasEvalResult = true;
   }
 
-  @Override
-  public void clearEvalResult() {
-    evalResult = null;
-    hasEvalResult = false;
-
-    dict.forEach(
-      (key, value) -> {
-        if (key instanceof EvalResultHolder) {
-          ((EvalResultHolder) key).clearEvalResult();
-        }
-        if (value instanceof EvalResultHolder) {
-          ((EvalResultHolder) value).clearEvalResult();
-        }
-      }
-    );
-  }
-
   @Override
   public boolean hasEvalResult() {
     return hasEvalResult;

src/main/java/com/hubspot/jinjava/el/ext/eager/EagerAstDot.java

@@ -72,13 +72,6 @@ public void setEvalResult(Object evalResult) {
     hasEvalResult = true;
   }
 
-  @Override
-  public void clearEvalResult() {
-    evalResult = null;
-    hasEvalResult = false;
-    base.clearEvalResult();
-  }
-
   @Override
   public boolean hasEvalResult() {
     return hasEvalResult;

src/main/java/com/hubspot/jinjava/el/ext/eager/EagerAstIdentifier.java

@@ -33,12 +33,6 @@ public void setEvalResult(Object evalResult) {
     hasEvalResult = true;
   }
 
-  @Override
-  public void clearEvalResult() {
-    evalResult = null;
-    hasEvalResult = false;
-  }
-
   @Override
   public boolean hasEvalResult() {
     return hasEvalResult;

src/main/java/com/hubspot/jinjava/el/ext/eager/EagerAstList.java

@@ -35,15 +35,6 @@ public void setEvalResult(Object evalResult) {
     hasEvalResult = true;
   }
 
-  @Override
-  public void clearEvalResult() {
-    evalResult = null;
-    hasEvalResult = false;
-    for (int i = 0; i < elements.getCardinality(); i++) {
-      ((EvalResultHolder) elements.getChild(i)).clearEvalResult();
-    }
-  }
-
   @Override
   public boolean hasEvalResult() {
     return hasEvalResult;

src/main/java/com/hubspot/jinjava/el/ext/eager/EagerAstMacroFunction.java

@@ -90,16 +90,6 @@ public void setEvalResult(Object evalResult) {
     hasEvalResult = true;
   }
 
-  @Override
-  public void clearEvalResult() {
-    evalResult = null;
-    hasEvalResult = false;
-    params.clearEvalResult();
-    for (int i = 0; i < ((AstParameters) params).getCardinality(); i++) {
-      ((EvalResultHolder) ((AstParameters) params).getChild(i)).clearEvalResult();
-    }
-  }
-
   @Override
   public boolean hasEvalResult() {
     return hasEvalResult;

src/main/java/com/hubspot/jinjava/el/ext/eager/EagerAstMethod.java

@@ -59,14 +59,6 @@ public void setEvalResult(Object evalResult) {
     hasEvalResult = true;
   }
 
-  @Override
-  public void clearEvalResult() {
-    evalResult = null;
-    hasEvalResult = false;
-    property.clearEvalResult();
-    params.clearEvalResult();
-  }
-
   @Override
   public boolean hasEvalResult() {
     return hasEvalResult;

src/main/java/com/hubspot/jinjava/el/ext/eager/EagerAstNamedParameter.java

@@ -65,13 +65,6 @@ public void setEvalResult(Object evalResult) {
     hasEvalResult = true;
   }
 
-  @Override
-  public void clearEvalResult() {
-    evalResult = null;
-    hasEvalResult = false;
-    value.clearEvalResult();
-  }
-
   @Override
   public boolean hasEvalResult() {
     return hasEvalResult;