Avoid keeping string around in the leaf nodes. It is not worth the space.

2 files changed, 1 insertions, 5 deletions

diff --git a/searchlib/src/main/java/com/yahoo/searchlib/rankingexpression/rule/ConstantNode.java b/searchlib/src/main/java/com/yahoo/searchlib/rankingexpression/rule/ConstantNode.java
index f682826997c..68134cc85b4 100755
--- a/searchlib/src/main/java/com/yahoo/searchlib/rankingexpression/rule/ConstantNode.java
+++ b/searchlib/src/main/java/com/yahoo/searchlib/rankingexpression/rule/ConstantNode.java
@@ -16,8 +16,6 @@ import java.util.Deque;
  */
 public final class ConstantNode extends ExpressionNode {
 
-    private final String sourceImage;
-
     private final Value value;
 
     public ConstantNode(Value value) {
@@ -33,7 +31,6 @@ public final class ConstantNode extends ExpressionNode {
     public ConstantNode(Value value, String sourceImage) {
         value.freeze();
         this.value = value;
-        this.sourceImage = sourceImage;
     }
 
     public Value getValue() { return value; }
@@ -45,7 +42,6 @@ public final class ConstantNode extends ExpressionNode {
 
     /** Returns the string which created this, or the value.toString() if not known */
     public String sourceString() {
-        if (sourceImage != null) return sourceImage;
         return value.toString();
     }
 
diff --git a/searchlib/src/main/javacc/RankingExpressionParser.jj b/searchlib/src/main/javacc/RankingExpressionParser.jj
index aab56435e64..61e35647b89 100755
--- a/searchlib/src/main/javacc/RankingExpressionParser.jj
+++ b/searchlib/src/main/javacc/RankingExpressionParser.jj
@@ -851,7 +851,7 @@ ConstantNode constantPrimitive() :
     ( <INTEGER> { value = token.image; } |
       <FLOAT> { value = token.image; } |
       <STRING> { value = token.image; } )
-    { return new ConstantNode(Value.parse(sign + value),sign + value); }
+    { return new ConstantNode(Value.parse(sign + value)); }
 }
 
 Value primitiveValue() :