Migrate from objc/cocoa to objc2

.changes/use-objc2.md

@@ -0,0 +1,7 @@
+---
+"muda": patch
+---
+
+Use `objc2` internally, leading to much better memory safety.
+
+The crate will panic now if used from a thread that is not the main thread.

Cargo.toml

@@ -44,9 +44,10 @@ features = [
 gtk = "0.18"
 libxdo = { version = "0.6.0", optional = true }
 
-[target."cfg(target_os = \"macos\")".dependencies]
-cocoa = "0.26"
-objc = "0.2"
+[target.'cfg(target_os = "macos")'.dependencies]
+objc2 = "0.5.2"
+objc2-foundation = { version = "0.2.2", features = ["all"] }
+objc2-app-kit = { version = "0.2.2", features = ["all"] }
 png = "0.17"
 
 [dev-dependencies]

src/lib.rs

@@ -14,6 +14,9 @@
 //!
 //! # Platform-specific notes:
 //!
+//! - On macOS, menus can only be used from the main thread, and most
+//!   functionality will panic if you try to use it from any other thread.
+//!
 //! - On Windows, accelerators don't work unless the win32 message loop calls
 //!   [`TranslateAcceleratorW`](https://docs.rs/windows-sys/latest/windows_sys/Win32/UI/WindowsAndMessaging/fn.TranslateAcceleratorW.html).
 //!   See [`Menu::init_for_hwnd`](https://docs.rs/muda/latest/x86_64-pc-windows-msvc/muda/struct.Menu.html#method.init_for_hwnd) for more details
@@ -140,10 +143,6 @@ mod menu_id;
 mod platform_impl;
 mod util;
 
-#[cfg(target_os = "macos")]
-#[macro_use]
-extern crate objc;
-
 pub use about_metadata::AboutMetadata;
 pub use builders::*;
 pub use dpi;

src/platform_impl/macos/accelerator.rs

@@ -2,8 +2,8 @@
 // SPDX-License-Identifier: Apache-2.0
 // SPDX-License-Identifier: MIT
 
-use cocoa::appkit::NSEventModifierFlags;
 use keyboard_types::{Code, Modifiers};
+use objc2_app_kit::NSEventModifierFlags;
 
 use crate::accelerator::{Accelerator, AcceleratorParseError};
 
@@ -112,16 +112,16 @@ impl Accelerator {
         let mods: Modifiers = self.mods;
         let mut flags = NSEventModifierFlags::empty();
         if mods.contains(Modifiers::SHIFT) {
-            flags.insert(NSEventModifierFlags::NSShiftKeyMask);
+            flags.insert(NSEventModifierFlags::NSEventModifierFlagShift);
         }
         if mods.contains(Modifiers::SUPER) {
-            flags.insert(NSEventModifierFlags::NSCommandKeyMask);
+            flags.insert(NSEventModifierFlags::NSEventModifierFlagCommand);
         }
         if mods.contains(Modifiers::ALT) {
-            flags.insert(NSEventModifierFlags::NSAlternateKeyMask);
+            flags.insert(NSEventModifierFlags::NSEventModifierFlagOption);
         }
         if mods.contains(Modifiers::CONTROL) {
-            flags.insert(NSEventModifierFlags::NSControlKeyMask);
+            flags.insert(NSEventModifierFlags::NSEventModifierFlagControl);
         }
         flags
     }

src/platform_impl/macos/icon.rs

@@ -2,6 +2,10 @@
 // SPDX-License-Identifier: Apache-2.0
 // SPDX-License-Identifier: MIT
 
+use objc2::{rc::Retained, ClassType};
+use objc2_app_kit::NSImage;
+use objc2_foundation::{CGFloat, NSData, NSSize};
+
 use crate::icon::{BadIcon, RgbaIcon};
 use std::io::Cursor;
 
@@ -33,36 +37,26 @@ impl PlatformIcon {
         png
     }
 
-    pub unsafe fn to_nsimage(&self, fixed_height: Option<f64>) -> cocoa::base::id {
-        use cocoa::{
-            appkit::NSImage,
-            base::nil,
-            foundation::{NSData, NSSize},
-        };
-
+    pub fn to_nsimage(&self, fixed_height: Option<f64>) -> Retained<NSImage> {
         let (width, height) = self.get_size();
         let icon = self.to_png();
 
         let (icon_width, icon_height) = match fixed_height {
             Some(fixed_height) => {
-                let icon_height: f64 = fixed_height;
-                let icon_width: f64 = (width as f64) / (height as f64 / icon_height);
+                let icon_height: CGFloat = fixed_height as CGFloat;
+                let icon_width: CGFloat = (width as CGFloat) / (height as CGFloat / icon_height);
 
                 (icon_width, icon_height)
             }
 
-            None => (width as f64, height as f64),
+            None => (width as CGFloat, height as CGFloat),
         };
 
-        let nsdata = NSData::dataWithBytes_length_(
-            nil,
-            icon.as_ptr() as *const std::os::raw::c_void,
-            icon.len() as u64,
-        );
+        let nsdata = NSData::with_bytes(&icon);
 
-        let nsimage = NSImage::initWithData_(NSImage::alloc(nil), nsdata);
+        let nsimage = NSImage::initWithData(NSImage::alloc(), &nsdata).unwrap();
         let new_size = NSSize::new(icon_width, icon_height);
-        let _: () = msg_send![nsimage, setSize: new_size];
+        unsafe { nsimage.setSize(new_size) };
 
         nsimage
     }