Remove the legacy createInterpretable(CheckedExpr)

PiperOrigin-RevId: 695519680
google · Nov 12, 2024 · 3cb263a · 3cb263a
1 parent c7a95d5
commit 3cb263a
Show file tree

Hide file tree

Showing 3 changed files with 0 additions and 21 deletions.
diff --git a/runtime/src/main/java/dev/cel/runtime/BUILD.bazel b/runtime/src/main/java/dev/cel/runtime/BUILD.bazel
@@ -45,7 +45,6 @@ java_library(
     ],
     deps = [
         ":runtime_helper",
-        "//:auto_value",
         "//common",
         "//common:error_codes",
         "//common:options",
@@ -85,7 +84,6 @@ java_library(
         "//common:error_codes",
         "//common:features",
         "//common:options",
-        "//common:proto_ast",
         "//common:runtime_exception",
         "//common/annotations",
         "//common/ast",

diff --git a/runtime/src/main/java/dev/cel/runtime/DefaultInterpreter.java b/runtime/src/main/java/dev/cel/runtime/DefaultInterpreter.java
@@ -14,7 +14,6 @@
 
 package dev.cel.runtime;
 
-import dev.cel.expr.CheckedExpr;
 import dev.cel.expr.Value;
 import com.google.auto.value.AutoValue;
 import com.google.common.base.Preconditions;
@@ -24,7 +23,6 @@
 import dev.cel.common.CelAbstractSyntaxTree;
 import dev.cel.common.CelErrorCode;
 import dev.cel.common.CelOptions;
-import dev.cel.common.CelProtoAbstractSyntaxTree;
 import dev.cel.common.annotations.Internal;
 import dev.cel.common.ast.CelConstant;
 import dev.cel.common.ast.CelExpr;
@@ -122,12 +120,6 @@ public DefaultInterpreter(
     this.celOptions = celOptions;
   }
 
-  @Override
-  @Deprecated
-  public Interpretable createInterpretable(CheckedExpr checkedExpr) {
-    return createInterpretable(CelProtoAbstractSyntaxTree.fromCheckedExpr(checkedExpr).getAst());
-  }
-
   @Override
   public Interpretable createInterpretable(CelAbstractSyntaxTree ast) {
     return new DefaultInterpretable(typeProvider, dispatcher, ast, celOptions);

diff --git a/runtime/src/main/java/dev/cel/runtime/Interpreter.java b/runtime/src/main/java/dev/cel/runtime/Interpreter.java
@@ -14,7 +14,6 @@
 
 package dev.cel.runtime;
 
-import dev.cel.expr.CheckedExpr;
 import javax.annotation.concurrent.ThreadSafe;
 import dev.cel.common.CelAbstractSyntaxTree;
 import dev.cel.common.annotations.Internal;
@@ -28,16 +27,6 @@
 @Internal
 public interface Interpreter {
 
-  /**
-   * Creates an interpretable for the given expression.
-   *
-   * <p>This method may run pre-processing and partial evaluation of the expression it gets passed.
-   *
-   * @deprecated Use {@link #createInterpretable(CelAbstractSyntaxTree)} instead.
-   */
-  @Deprecated
-  Interpretable createInterpretable(CheckedExpr checkedExpr) throws InterpreterException;
-
   /**
    * Creates an interpretable for the given expression.
    *