Remove redundant FunctionBindingCreator interface, consolidate with CelStandardOverload

PiperOrigin-RevId: 826652181

Author: l46kok

runtime/src/main/java/dev/cel/runtime/CelStandardFunctions.java

@@ -156,16 +156,16 @@ public enum AddOverload implements CelStandardOverload {
         (celOptions, runtimeEquality) ->
             CelFunctionBinding.from("add_list", List.class, List.class, RuntimeHelpers::concat));
 
-    private final FunctionBindingCreator bindingCreator;
+    private final CelStandardOverload standardOverload;
 
     @Override
     public CelFunctionBinding newFunctionBinding(
         CelOptions celOptions, RuntimeEquality runtimeEquality) {
-      return bindingCreator.create(celOptions, runtimeEquality);
+      return standardOverload.newFunctionBinding(celOptions, runtimeEquality);
     }
 
-    AddOverload(FunctionBindingCreator bindingCreator) {
-      this.bindingCreator = bindingCreator;
+    AddOverload(CelStandardOverload standardOverload) {
+      this.standardOverload = standardOverload;
     }
   }
 

runtime/src/main/java/dev/cel/runtime/standard/AddOperator.java

@@ -72,17 +72,17 @@ public enum BoolOverload implements CelStandardOverload {
                   }
                 }));
 
-    private final FunctionBindingCreator bindingCreator;
+    private final CelStandardOverload standardOverload;
     ;
 
     @Override
     public CelFunctionBinding newFunctionBinding(
         CelOptions celOptions, RuntimeEquality runtimeEquality) {
-      return bindingCreator.create(celOptions, runtimeEquality);
+      return standardOverload.newFunctionBinding(celOptions, runtimeEquality);
     }
 
-    BoolOverload(FunctionBindingCreator bindingCreator) {
-      this.bindingCreator = bindingCreator;
+    BoolOverload(CelStandardOverload standardOverload) {
+      this.standardOverload = standardOverload;
     }
   }
 

runtime/src/main/java/dev/cel/runtime/standard/BoolFunction.java

@@ -61,17 +61,17 @@ public enum BytesOverload implements CelStandardOverload {
         }),
     ;
 
-    private final FunctionBindingCreator bindingCreator;
+    private final CelStandardOverload standardOverload;
     ;
 
     @Override
     public CelFunctionBinding newFunctionBinding(
         CelOptions celOptions, RuntimeEquality runtimeEquality) {
-      return bindingCreator.create(celOptions, runtimeEquality);
+      return standardOverload.newFunctionBinding(celOptions, runtimeEquality);
     }
 
-    BytesOverload(FunctionBindingCreator bindingCreator) {
-      this.bindingCreator = bindingCreator;
+    BytesOverload(CelStandardOverload standardOverload) {
+      this.standardOverload = standardOverload;
     }
   }
 

runtime/src/main/java/dev/cel/runtime/standard/BytesFunction.java

@@ -25,15 +25,9 @@
  * overload.
  */
 @Immutable
+@FunctionalInterface
 public interface CelStandardOverload {
 
   /** Constructs a new {@link CelFunctionBinding} for this CEL standard overload. */
   CelFunctionBinding newFunctionBinding(CelOptions celOptions, RuntimeEquality runtimeEquality);
-
-  /** TODO: To be removed in the upcoming CL */
-  @FunctionalInterface
-  @Immutable
-  interface FunctionBindingCreator {
-    CelFunctionBinding create(CelOptions celOptions, RuntimeEquality runtimeEquality);
-  }
 }

runtime/src/main/java/dev/cel/runtime/standard/CelStandardOverload.java

@@ -44,16 +44,16 @@ public enum ContainsOverload implements CelStandardOverload {
                 "contains_string", String.class, String.class, String::contains)),
     ;
 
-    private final FunctionBindingCreator bindingCreator;
+    private final CelStandardOverload standardOverload;
 
     @Override
     public CelFunctionBinding newFunctionBinding(
         CelOptions celOptions, RuntimeEquality runtimeEquality) {
-      return bindingCreator.create(celOptions, runtimeEquality);
+      return standardOverload.newFunctionBinding(celOptions, runtimeEquality);
     }
 
-    ContainsOverload(FunctionBindingCreator bindingCreator) {
-      this.bindingCreator = bindingCreator;
+    ContainsOverload(CelStandardOverload standardOverload) {
+      this.standardOverload = standardOverload;
     }
   }
 

runtime/src/main/java/dev/cel/runtime/standard/ContainsFunction.java

@@ -76,16 +76,16 @@ public enum DivideOverload implements CelStandardOverload {
                 (Long x, Long y) -> RuntimeHelpers.uint64Divide(x, y, celOptions));
           }
         });
-    private final FunctionBindingCreator bindingCreator;
+    private final CelStandardOverload standardOverload;
 
     @Override
     public CelFunctionBinding newFunctionBinding(
         CelOptions celOptions, RuntimeEquality runtimeEquality) {
-      return bindingCreator.create(celOptions, runtimeEquality);
+      return standardOverload.newFunctionBinding(celOptions, runtimeEquality);
     }
 
-    DivideOverload(FunctionBindingCreator bindingCreator) {
-      this.bindingCreator = bindingCreator;
+    DivideOverload(CelStandardOverload standardOverload) {
+      this.standardOverload = standardOverload;
     }
   }
 

runtime/src/main/java/dev/cel/runtime/standard/DivideOperator.java

@@ -73,16 +73,16 @@ public enum DoubleOverload implements CelStandardOverload {
         }),
     ;
 
-    private final FunctionBindingCreator bindingCreator;
+    private final CelStandardOverload standardOverload;
 
     @Override
     public CelFunctionBinding newFunctionBinding(
         CelOptions celOptions, RuntimeEquality runtimeEquality) {
-      return bindingCreator.create(celOptions, runtimeEquality);
+      return standardOverload.newFunctionBinding(celOptions, runtimeEquality);
     }
 
-    DoubleOverload(FunctionBindingCreator bindingCreator) {
-      this.bindingCreator = bindingCreator;
+    DoubleOverload(CelStandardOverload standardOverload) {
+      this.standardOverload = standardOverload;
     }
   }
 

runtime/src/main/java/dev/cel/runtime/standard/DoubleFunction.java

@@ -74,16 +74,16 @@ public enum DurationOverload implements CelStandardOverload {
                 })),
     ;
 
-    private final FunctionBindingCreator bindingCreator;
+    private final CelStandardOverload standardOverload;
 
     @Override
     public CelFunctionBinding newFunctionBinding(
         CelOptions celOptions, RuntimeEquality runtimeEquality) {
-      return bindingCreator.create(celOptions, runtimeEquality);
+      return standardOverload.newFunctionBinding(celOptions, runtimeEquality);
     }
 
-    DurationOverload(FunctionBindingCreator bindingCreator) {
-      this.bindingCreator = bindingCreator;
+    DurationOverload(CelStandardOverload standardOverload) {
+      this.standardOverload = standardOverload;
     }
   }
 

runtime/src/main/java/dev/cel/runtime/standard/DurationFunction.java

@@ -43,16 +43,16 @@ public enum DynOverload implements CelStandardOverload {
         (celOptions, runtimeEquality) ->
             CelFunctionBinding.from("to_dyn", Object.class, (Object arg) -> arg));
 
-    private final FunctionBindingCreator bindingCreator;
+    private final CelStandardOverload standardOverload;
 
     @Override
     public CelFunctionBinding newFunctionBinding(
         CelOptions celOptions, RuntimeEquality runtimeEquality) {
-      return bindingCreator.create(celOptions, runtimeEquality);
+      return standardOverload.newFunctionBinding(celOptions, runtimeEquality);
     }
 
-    DynOverload(FunctionBindingCreator bindingCreator) {
-      this.bindingCreator = bindingCreator;
+    DynOverload(CelStandardOverload standardOverload) {
+      this.standardOverload = standardOverload;
     }
   }