Fix memory management in header-translator

crates/header-translator/src/cache.rs

@@ -29,7 +29,7 @@ impl ClassCache {
     fn all_methods_data(&self) -> impl Iterator<Item = (bool, &str)> {
         self.to_emit
             .iter()
-            .flat_map(|cache| cache.methods.iter().map(|m| (m.is_class, &*m.selector)))
+            .flat_map(|cache| cache.methods.iter().map(|m| m.id()))
     }
 }
 
@@ -147,9 +147,7 @@ impl<'a> Cache<'a> {
                                 let mut methods: Vec<_> = cache
                                     .methods
                                     .iter()
-                                    .filter(|method| {
-                                        !seen_methods.contains(&(method.is_class, &method.selector))
-                                    })
+                                    .filter(|method| !seen_methods.contains(&method.id()))
                                     .filter_map(|method| {
                                         method.clone().update(ClassData::get_method_data(
                                             data,

crates/header-translator/src/expr.rs

@@ -4,8 +4,9 @@ use std::fmt::Write;
 use clang::token::TokenKind;
 use clang::{Entity, EntityKind, EntityVisitResult};
 
+use crate::context::Context;
 use crate::immediate_children;
-use crate::unexposed_macro::UnexposedMacro;
+use crate::unexposed_attr::UnexposedAttr;
 
 #[derive(Clone, Debug, PartialEq)]
 pub enum Expr {
@@ -36,7 +37,7 @@ impl Expr {
         }
     }
 
-    pub fn parse_enum_constant(entity: &Entity<'_>) -> Option<Self> {
+    pub fn parse_enum_constant(entity: &Entity<'_>, context: &Context<'_>) -> Option<Self> {
         let mut declaration_references = Vec::new();
 
         entity.visit_children(|entity, _parent| {
@@ -51,8 +52,8 @@ impl Expr {
 
         immediate_children(entity, |entity, _span| match entity.get_kind() {
             EntityKind::UnexposedAttr => {
-                if let Some(macro_) = UnexposedMacro::parse(&entity) {
-                    panic!("parsed macro in expr: {macro_:?}, {entity:?}");
+                if let Some(attr) = UnexposedAttr::parse(&entity, context) {
+                    error!(?attr, "unknown attribute");
                 }
             }
             _ => {

crates/header-translator/src/lib.rs

@@ -24,10 +24,9 @@ mod library;
 mod method;
 mod objc2_utils;
 mod output;
-mod property;
 mod rust_type;
 mod stmt;
-mod unexposed_macro;
+mod unexposed_attr;
 
 pub use self::cache::Cache;
 pub use self::config::Config;