enum: Remove an unnecessary field

Reviewed By: IanChilds

Differential Revision: D73726003

fbshipit-source-id: 6268f94d42ebf4952221472eb81e2f4c0912a382

Author: JakobDegen

starlark/src/values/types/enumeration/enum_type.rs

@@ -61,7 +61,6 @@ use crate::values::ValueTyped;
 use crate::values::enumeration::EnumValue;
 use crate::values::enumeration::matcher::EnumTypeMatcher;
 use crate::values::enumeration::ty_enum_type::TyEnumData;
-use crate::values::enumeration::value::EnumValueGen;
 use crate::values::function::FUNCTION_TYPE;
 use crate::values::index::convert_index;
 use crate::values::list::AllocList;
@@ -340,16 +339,6 @@ where
             )?);
             Ok(Arc::new(TyEnumData {
                 name: variable_name.to_owned(),
-                variants: self
-                    .elements()
-                    .iter()
-                    .map(|(_, enum_value)| {
-                        let enum_value: &EnumValueGen<_> =
-                            EnumValue::from_value(enum_value.to_value())
-                                .expect("known to be enum value");
-                        Ty::of_value(enum_value.value)
-                    })
-                    .collect(),
                 id: self.id,
                 ty_enum_value,
                 ty_enum_type,

starlark/src/values/types/enumeration/ty_enum_type.rs

@@ -25,8 +25,6 @@ use crate::values::types::type_instance_id::TypeInstanceId;
 pub struct TyEnumData {
     /// Name of the enum type.
     pub(crate) name: String,
-    /// Types of variants.
-    pub(crate) variants: Vec<Ty>,
     /// Globally unique id of the enum type.
     // Id must be last so `Ord` is deterministic.
     pub(crate) id: TypeInstanceId,
@@ -48,6 +46,5 @@ impl std::hash::Hash for TyEnumData {
     fn hash<H: std::hash::Hasher>(&self, state: &mut H) {
         // Do not hash `id` because hashing should be deterministic.
         self.name.hash(state);
-        self.variants.hash(state);
     }
 }