Use sendability attributes to implement Send + Sync

Author: madsmtm

crates/header-translator/src/stmt.rs

@@ -1319,7 +1319,7 @@ impl fmt::Display for Stmt {
                 derives,
                 mutability,
                 skipped,
-                sendable: _,
+                sendable,
                 mainthreadonly: _,
             } => {
                 if *skipped {
@@ -1429,6 +1429,20 @@ impl fmt::Display for Stmt {
                 }
                 writeln!(f, "    }}")?;
                 writeln!(f, ");")?;
+
+                if *sendable && generics.is_empty() {
+                    writeln!(f)?;
+                    if let Some(feature) = &main_feature_gate {
+                        writeln!(f, "    #[cfg(feature = \"{feature}\")]")?;
+                    }
+                    writeln!(f, "unsafe impl Send for {} {{}}", id.name)?;
+
+                    writeln!(f)?;
+                    if let Some(feature) = &main_feature_gate {
+                        writeln!(f, "    #[cfg(feature = \"{feature}\")]")?;
+                    }
+                    writeln!(f, "unsafe impl Sync for {} {{}}", id.name)?;
+                }
             }
             Self::Methods {
                 cls,
@@ -1608,6 +1622,15 @@ impl fmt::Display for Stmt {
                         write!(f, "{}", protocol.path())?;
                     }
                 }
+                // TODO
+                // if *required_sendable {
+                //     if protocols.is_empty() {
+                //         write!(f, ": ")?;
+                //     } else {
+                //         write!(f, "+ ")?;
+                //     }
+                //     write!(f, "Send + Sync")?;
+                // }
                 writeln!(f, " {{")?;
 
                 for method in methods {
@@ -1649,7 +1672,7 @@ impl fmt::Display for Stmt {
                 availability,
                 boxable: _,
                 fields,
-                sendable: _,
+                sendable,
             } => {
                 writeln!(f, "extern_struct!(")?;
                 if let Some(encoding_name) = encoding_name {
@@ -1667,14 +1690,22 @@ impl fmt::Display for Stmt {
                 }
                 writeln!(f, "    }}")?;
                 writeln!(f, ");")?;
+
+                if let Some(true) = sendable {
+                    writeln!(f)?;
+                    writeln!(f, "unsafe impl Send for {} {{}}", id.name)?;
+
+                    writeln!(f)?;
+                    writeln!(f, "unsafe impl Sync for {} {{}}", id.name)?;
+                }
             }
             Self::EnumDecl {
                 id,
                 availability,
                 ty,
                 kind,
                 variants,
-                sendable: _,
+                sendable,
             } => {
                 let macro_name = match kind {
                     None => "extern_enum",
@@ -1698,6 +1729,16 @@ impl fmt::Display for Stmt {
                 }
                 writeln!(f, "    }}")?;
                 writeln!(f, ");")?;
+
+                if let Some(true) = sendable {
+                    if let Some(name) = &id.name {
+                        writeln!(f)?;
+                        writeln!(f, "unsafe impl Send for {name} {{}}")?;
+
+                        writeln!(f)?;
+                        writeln!(f, "unsafe impl Sync for {name} {{}}")?;
+                    }
+                }
             }
             Self::VarDecl {
                 id,

crates/icrate/CHANGELOG.md

@@ -12,6 +12,8 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
 
 ### Added
 * Added `MainThreadMarker` `From` implementation for `MainThreadOnly` types.
+* Added `Send` and `Sync` implementations for a bunch more types (same as the
+  ones Swift marks as `@Sendable`).
 
 ### Changed
 * Moved the `ns_string!` macro to `icrate::Foundation::ns_string`. The old

crates/icrate/src/additions/Foundation/error.rs

@@ -9,10 +9,6 @@ use crate::Foundation::{
     self, NSError, NSErrorDomain, NSErrorUserInfoKey, NSInteger, NSLocalizedDescriptionKey,
 };
 
-// SAFETY: Error objects are immutable data containers.
-unsafe impl Sync for NSError {}
-unsafe impl Send for NSError {}
-
 impl UnwindSafe for NSError {}
 impl RefUnwindSafe for NSError {}
 

crates/icrate/src/additions/Foundation/number.rs

@@ -20,11 +20,6 @@ use objc2::encode::Encoding;
 use crate::common::*;
 use crate::Foundation::{CGFloat, NSNumber};
 
-// SAFETY: `NSNumber` is a wrapper around an integer/float/bool, and it is
-// immutable.
-unsafe impl Sync for NSNumber {}
-unsafe impl Send for NSNumber {}
-
 impl UnwindSafe for NSNumber {}
 impl RefUnwindSafe for NSNumber {}
 

crates/icrate/src/additions/Foundation/uuid.rs

@@ -5,10 +5,6 @@ use core::panic::{RefUnwindSafe, UnwindSafe};
 use crate::common::*;
 use crate::Foundation::{self, UuidBytes, NSUUID};
 
-// SAFETY: `NSUUID` is immutable.
-unsafe impl Sync for NSUUID {}
-unsafe impl Send for NSUUID {}
-
 impl UnwindSafe for NSUUID {}
 impl RefUnwindSafe for NSUUID {}