fix: no need to box interfaces

MedHeikelBouzayene · Jan 5, 2025 · 5436d47 · 5436d47
1 parent 2425a54
commit 5436d47
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 11 deletions.
diff --git a/src/core/blueprint/index.rs b/src/core/blueprint/index.rs
@@ -40,16 +40,14 @@ impl Index {
         matches!(def, Some(Definition::Scalar(_))) || scalar::Scalar::is_predefined(type_name)
     }
 
-    pub fn get_interfaces(&self) -> Box<HashSet<String>> {
-        Box::new(
-            self.map
-                .iter()
-                .filter_map(|(_, (def, _))| match def {
-                    Definition::Interface(i) => Some(i.name.to_owned()),
-                    _ => None,
-                })
-                .collect(),
-        )
+    pub fn get_interfaces(&self) -> HashSet<String> {
+        self.map
+            .iter()
+            .filter_map(|(_, (def, _))| match def {
+                Definition::Interface(i) => Some(i.name.to_owned()),
+                _ => None,
+            })
+            .collect()
     }
 
     pub fn type_is_enum(&self, type_name: &str) -> bool {

diff --git a/src/core/jit/builder.rs b/src/core/jit/builder.rs
@@ -364,7 +364,7 @@ impl<'a> Builder<'a> {
             operation.ty,
             self.index.clone(),
             is_introspection_query,
-            Some(*self.index.get_interfaces()),
+            Some(self.index.get_interfaces()),
         );
         Ok(plan)
     }