From 0e4c6391cd2278c910fbe01f1184d7fc93ba1993 Mon Sep 17 00:00:00 2001
From: Henning Baldersheim <balder@yahoo-inc.com>
Date: Tue, 14 Sep 2021 16:09:24 +0200
Subject: Use a single cached object to make logic simpler and only update on
 object.

---
 .../rankingexpression/rule/FunctionReferenceContext.java          | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

(limited to 'searchlib/src/main/java/com/yahoo/searchlib/rankingexpression/rule/FunctionReferenceContext.java')

diff --git a/searchlib/src/main/java/com/yahoo/searchlib/rankingexpression/rule/FunctionReferenceContext.java b/searchlib/src/main/java/com/yahoo/searchlib/rankingexpression/rule/FunctionReferenceContext.java
index 6717bec0258..506884d15b4 100644
--- a/searchlib/src/main/java/com/yahoo/searchlib/rankingexpression/rule/FunctionReferenceContext.java
+++ b/searchlib/src/main/java/com/yahoo/searchlib/rankingexpression/rule/FunctionReferenceContext.java
@@ -38,12 +38,16 @@ public class FunctionReferenceContext {
 
     /** Create a context for a single serialization task */
     public FunctionReferenceContext(Map<String, ExpressionFunction> functions) {
-        this(functions.values());
+        this(functions, null);
     }
 
     /** Create a context for a single serialization task */
     public FunctionReferenceContext(Map<String, ExpressionFunction> functions, Map<String, String> bindings) {
-        this.functions = ImmutableMap.copyOf(functions);
+        this(ImmutableMap.copyOf(functions), bindings);
+    }
+
+    protected FunctionReferenceContext(ImmutableMap<String, ExpressionFunction> functions, Map<String, String> bindings) {
+        this.functions = functions;
         if (bindings != null)
             this.bindings.putAll(bindings);
     }
-- 
cgit v1.2.3