Delete AstImpl, everything should just use Ast directly now.

PiperOrigin-RevId: 800548853

Author: jnthntatum

common/ast.h

@@ -27,10 +27,6 @@
 
 namespace cel {
 
-namespace ast_internal {
-class AstImpl;
-}  // namespace ast_internal
-
 // In memory representation of a CEL abstract syntax tree.
 //
 // If AST inspection or manipulation is needed, prefer to use an existing tool
@@ -42,13 +38,11 @@ class AstImpl;
 //
 // To create a new instance from a protobuf representation, use the conversion
 // utilities in `common/ast_proto.h`.
-class Ast {
+class Ast final {
  public:
   using ReferenceMap = absl::flat_hash_map<int64_t, Reference>;
   using TypeMap = absl::flat_hash_map<int64_t, TypeSpec>;
 
-  virtual ~Ast() = default;
-
   Ast() : is_checked_(false) {}
 
   Ast(Expr expr, SourceInfo source_info)
@@ -142,8 +136,6 @@ class Ast {
   }
 
  private:
-  friend class ast_internal::AstImpl;
-
   Expr root_expr_;
   SourceInfo source_info_;
   ReferenceMap reference_map_;

common/ast/BUILD

@@ -73,17 +73,6 @@ cc_test(
     ],
 )
 
-cc_library(
-    name = "ast_impl",
-    hdrs = ["ast_impl.h"],
-    deps = [
-        ":expr",
-        ":metadata",
-        "//common:ast",
-        "//common:expr",
-    ],
-)
-
 cc_library(
     name = "expr",
     hdrs = [