From 7f5085cbc75e3dfcb655696b133c824e6310a73d Mon Sep 17 00:00:00 2001
From: Adam Chalmers <adam.chalmers@zoo.dev>
Date: Fri, 13 Dec 2024 16:22:13 -0600
Subject: [PATCH] ExtrusionFaceInfo::list_faces should be a method

---
 modeling-cmds/src/shared.rs | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/modeling-cmds/src/shared.rs b/modeling-cmds/src/shared.rs
index 94d608b5..2fa1dada 100644
--- a/modeling-cmds/src/shared.rs
+++ b/modeling-cmds/src/shared.rs
@@ -842,8 +842,8 @@ fn z_axis() -> Point3d<f64> {
 impl ExtrudedFaceInfo {
     /// Converts from the representation used in the Extrude modeling command,
     /// to a flat representation.
-    pub fn list_faces(faces: ExtrudedFaceInfo) -> Vec<ExtrusionFaceInfo> {
-        let mut face_infos: Vec<_> = faces
+    pub fn list_faces(self) -> Vec<ExtrusionFaceInfo> {
+        let mut face_infos: Vec<_> = self
             .sides
             .into_iter()
             .map(|side| ExtrusionFaceInfo {
@@ -854,10 +854,10 @@ impl ExtrudedFaceInfo {
             .collect();
         face_infos.push(ExtrusionFaceInfo {
             curve_id: None,
-            face_id: Some(faces.top),
+            face_id: Some(self.top),
             cap: ExtrusionFaceCapType::Top,
         });
-        if let Some(bottom) = faces.bottom {
+        if let Some(bottom) = self.bottom {
             face_infos.push(ExtrusionFaceInfo {
                 curve_id: None,
                 face_id: Some(bottom),