From eb686e2c952e04da74b3e12c0bfa015ec4615e1d Mon Sep 17 00:00:00 2001
From: Jack Wills <32690432+mrjackwills@users.noreply.github.com>
Date: Thu, 14 Aug 2025 23:55:04 +0000
Subject: [PATCH] refactor: remove macos cfg conts functions

Zigbuild updated to rust 1.87
---
 src/app_data/container_state.rs | 24 ------------------------
 src/app_data/mod.rs             |  7 -------
 2 files changed, 31 deletions(-)

diff --git a/src/app_data/container_state.rs b/src/app_data/container_state.rs
index 97e9f42..ed605f0 100644
--- a/src/app_data/container_state.rs
+++ b/src/app_data/container_state.rs
@@ -27,14 +27,6 @@ impl From<&str> for ContainerId {
 }
 
 impl ContainerId {
-    // TODO remove this once zigbuild uses Rust v1.87.0
-    #[cfg(target_os = "macos")]
-    #[allow(clippy::missing_const_for_fn)]
-    pub fn get(&self) -> &str {
-        self.0.as_str()
-    }
-
-    #[cfg(not(target_os = "macos"))]
     pub const fn get(&self) -> &str {
         self.0.as_str()
     }
@@ -81,14 +73,6 @@ macro_rules! unit_struct {
         }
 
         impl $name {
-            #[cfg(target_os = "macos")]
-            #[allow(clippy::missing_const_for_fn)]
-            // TODO remove this once zigbuild uses Rust v1.87.0
-            pub fn get(&self) -> &str {
-                self.0.as_str()
-            }
-
-            #[cfg(not(target_os = "macos"))]
             pub const fn get(&self) -> &str {
                 self.0.as_str()
             }
@@ -706,14 +690,6 @@ impl Logs {
         self.lines.start();
     }
 
-    // // TODO remove this once zigbuild uses Rust v1.87.0
-    // #[cfg(target_os = "macos")]
-    // #[allow(clippy::missing_const_for_fn)]
-    // pub fn len(&self) -> usize {
-    //     self.logs.items.len()
-    // }
-
-    // #[cfg(not(target_os = "macos"))]
     pub const fn len(&self) -> usize {
         self.lines.items.len()
     }
diff --git a/src/app_data/mod.rs b/src/app_data/mod.rs
index a6e17af..0b97449 100644
--- a/src/app_data/mod.rs
+++ b/src/app_data/mod.rs
@@ -407,13 +407,6 @@ impl AppData {
 
     /// Container state methods
     /// Get the total number of none "hidden" containers
-    // TODO remove this once zigbuild uses Rust v1.87.0
-    #[cfg(target_os = "macos")]
-    pub fn get_container_len(&self) -> usize {
-        self.containers.items.len()
-    }
-
-    #[cfg(not(target_os = "macos"))]
     pub const fn get_container_len(&self) -> usize {
         self.containers.items.len()
     }