Various cleanups in dispatch2

- Add to CI.
- Update to 2021 edition.
- Appease Clippy.
- Support visionOS and watchOS.
- Align Cargo.toml metadata with the rest of the project.
- Fix typos.

Part of #77.

Author: madsmtm

.github/workflows/ci.yml

@@ -38,6 +38,7 @@ env:
   #
   # This excludes `header-translator`, `test-assembly`, `tests` and `test-ui`.
   PUBLIC_CRATES: >-
+    --package=dispatch2
     --package=block2
     --package=objc2
     --package=objc2-encode
@@ -113,12 +114,12 @@ jobs:
           target: armv7s-apple-ios
           build-std: true
           # `cc` works poorly when cross-compiling, so let's ignore `objc2-exception-helper` for now.
-          args: --package=block2 --package=objc2 --package=objc2-encode --package=objc2-proc-macros $FRAMEWORKS_IOS_17 $INTERESTING_FEATURES -Zbuild-std
+          args: --package=dispatch2 --package=block2 --package=objc2 --package=objc2-encode --package=objc2-proc-macros $FRAMEWORKS_IOS_17 $INTERESTING_FEATURES -Zbuild-std
         - name: visionOS Aarch64 simulator
           target: aarch64-apple-visionos-sim
           build-std: true
           # `cc` works poorly when cross-compiling, so let's ignore `objc2-exception-helper` for now.
-          args: --package=block2 --package=objc2 --package=objc2-encode --package=objc2-proc-macros $FRAMEWORKS_VISIONOS_1 $INTERESTING_FEATURES -Zbuild-std
+          args: --package=dispatch2 --package=block2 --package=objc2 --package=objc2-encode --package=objc2-proc-macros $FRAMEWORKS_VISIONOS_1 $INTERESTING_FEATURES -Zbuild-std
 
         - name: GNUStep + exceptions
           target: x86_64-unknown-linux-gnu
@@ -198,7 +199,7 @@ jobs:
         key: cargo-${{ github.job }}-${{ matrix.name }}-${{ hashFiles('**/Cargo.lock') }}
 
     - name: cargo check
-      run: cargo check --package=block2 --package=objc2 --package=objc2-encode --package=objc2-proc-macros $FRAMEWORKS_MACOS_14 --features=all
+      run: cargo check --package=dispatch2 --package=block2 --package=objc2 --package=objc2-encode --package=objc2-proc-macros $FRAMEWORKS_MACOS_14 --features=all
       env:
         RUSTFLAGS: "--codegen=debuginfo=0" # Removed --deny=warnings
 
@@ -657,6 +658,7 @@ jobs:
       RUNTIME_VERSION: gnustep-${{ matrix.libobjc2 }}
       # Exclude `objc2-exception-helper`, for some reason that can't be tested directly on GNUStep
       PUBLIC_CRATES: >-
+        --package=dispatch2
         --package=block2
         --package=objc2
         --package=objc2-encode

Cargo.lock

@@ -15,6 +15,9 @@ rust-version.workspace = true
 repository.workspace = true
 license = "MIT" # https://github.com/madsmtm/objc2/issues/23
 
+[lints]
+workspace = true
+
 [features]
 default = ["std"]
 

crates/block2/Cargo.toml

@@ -18,7 +18,7 @@ use objc2::encode::{EncodeArguments, EncodeReturn, Encoding};
 /// assert_eq!(block_signature_string::<(i32, f32), u8>(), "C16@?0i8f12");
 /// ```
 #[allow(unused)]
-pub fn block_signature_string<A, R>() -> CString
+pub(crate) fn block_signature_string<A, R>() -> CString
 where
     A: EncodeArguments,
     R: EncodeReturn,

crates/block2/src/encoding.rs

@@ -1,18 +1,42 @@
 [package]
 name = "dispatch2"
-version = "0.1.0"
-authors = ["Mary <[email protected]>"]
+version = "0.1.0" # Remember to update html_root_url in lib.rs
+description = "Bindings and wrappers for Apple's Grand Central Dispatch (GCD)"
+keywords = ["gcd", "macos", "ios", "dispatch", "libdispatch"]
+categories = [
+    "api-bindings",
+    "development-tools::ffi",
+    "os::macos-apis",
+    "external-ffi-bindings",
+]
+authors = ["Mads Marquart <[email protected]>", "Mary <[email protected]>"]
+edition.workspace = true
+rust-version.workspace = true
+repository.workspace = true
 license = "Apache-2.0 OR MIT"
-repository = "https://www.github.com/marysaka/dispatch2.git"
-homepage = "https://www.github.com/marysaka/dispatch2"
-description = "Bindings and wrappers for the Grand Central Dispatch (GCD)"
-keywords = ["gcd", "macOS", "iOS", "watchOS", "ipadOS"]
-categories = ["api-bindings", "development-tools::ffi", "os::macos-apis"]
-edition = "2021"
 
-exclude = [
-    ".github"
-]
+[lints]
+workspace = true
 
 [dependencies]
 libc = "0.2"
+
+[package.metadata.docs.rs]
+default-target = "aarch64-apple-darwin"
+features = []
+targets = [
+    "aarch64-apple-darwin",
+    "x86_64-apple-darwin",
+    "aarch64-apple-ios",
+    "x86_64-apple-ios",
+    "aarch64-apple-tvos",
+    "aarch64-apple-watchos",
+    "aarch64-apple-ios-macabi",
+    "x86_64-unknown-linux-gnu",
+    "i686-unknown-linux-gnu",
+]
+
+[package.metadata.release]
+shared-version = false
+tag-prefix = "dispatch"
+enable-features = []

crates/dispatch2/Cargo.toml

@@ -6,7 +6,7 @@
 - CI test on Windows using https://github.com/apple/swift-corelibs-libdispatch
 - Safe wrapper for ``dispatch_source_*`` + ``set_target_queue/activate/suspend/resume`` for it
 - Safe wrapper for ``dispatch_data_*``
-- Safe wrapper for ``dispatch_once_f`` (is that relevent?)
+- Safe wrapper for ``dispatch_once_f`` (is that relevant?)
 - Safe wrapper for ``dispatch_get_context/dispatch_set_context`` (quite impossible without big overhead => wrap dispatch object destructor to release the boxed value)
 - All blocks related bindings and ``dispatch_block_*`` functions with compat with ``block2`` on Apple platforms.
 - Integrate conversion from SystemTime to dispatch_time_t via dispatch_walltime and safe APIs using that.

crates/dispatch2/TODO.md

@@ -8,7 +8,7 @@ use core::ffi::{c_char, c_int, c_long, c_uint, c_ulong, c_void};
 macro_rules! create_opaque_type {
     ($type_name: ident, $typedef_name: ident) => {
         #[repr(C)]
-        #[derive(Debug)]
+        #[derive(Copy, Clone, Debug)]
         #[allow(missing_docs)]
         pub struct $type_name {
             /// opaque value
@@ -32,7 +32,7 @@ macro_rules! enum_with_val {
         }
 
         impl ::core::fmt::Debug for $ident {
-            fn fmt(&self, f: &mut ::core::fmt::Formatter) -> ::core::fmt::Result {
+            fn fmt(&self, f: &mut ::core::fmt::Formatter<'_>) -> ::core::fmt::Result {
                 match self {
                     $(&$ident::$variant => write!(f, "{}::{}", stringify!($ident), stringify!($variant)),)*
                     &$ident(v) => write!(f, "UNKNOWN({})", v),
@@ -191,14 +191,8 @@ enum_with_val! {
     }
 }
 
-#[cfg_attr(
-    any(target_os = "macos", target_os = "ios", target_os = "tvos"),
-    link(name = "System", kind = "dylib")
-)]
-#[cfg_attr(
-    not(any(target_os = "macos", target_os = "ios", target_os = "tvos")),
-    link(name = "dispatch", kind = "dylib")
-)]
+#[cfg_attr(target_vendor = "apple", link(name = "System", kind = "dylib"))]
+#[cfg_attr(not(target_vendor = "apple"), link(name = "dispatch", kind = "dylib"))]
 extern "C" {
     /// Increments the reference count (the retain count) of a dispatch object.
     pub fn dispatch_retain(object: dispatch_object_t);
@@ -391,7 +385,7 @@ extern "C" {
     ) -> dispatch_queue_attr_t;
     /// Creates a new dispatch queue to which you can submit blocks.
     #[cfg_attr(
-        any(target_os = "macos", target_os = "ios", target_os = "tvos"),
+        target_vendor = "apple",
         link_name = "dispatch_queue_create_with_target$V2"
     )]
     pub fn dispatch_queue_create_with_target(
@@ -459,18 +453,12 @@ extern "C" {
     pub fn dispatch_queue_get_specific(queue: dispatch_queue_t, key: *const c_void) -> *mut c_void;
     /// Returns the value for the key associated with the current dispatch queue.
     pub fn dispatch_get_specific(key: *const c_void) -> *mut c_void;
-    #[cfg_attr(
-        any(target_os = "macos", target_os = "ios", target_os = "tvos"),
-        link_name = "dispatch_assert_queue$V2"
-    )]
+    #[cfg_attr(target_vendor = "apple", link_name = "dispatch_assert_queue$V2")]
     /// Generates an assertion if the current block is not running on the specified dispatch queue.
     pub fn dispatch_assert_queue(queue: dispatch_queue_t);
     /// Generates an assertion if the current block is not running as a barrier on the specified dispatch queue.
     pub fn dispatch_assert_queue_barrier(queue: dispatch_queue_t);
-    #[cfg_attr(
-        any(target_os = "macos", target_os = "ios", target_os = "tvos"),
-        link_name = "dispatch_assert_queue_not$V2"
-    )]
+    #[cfg_attr(target_vendor = "apple", link_name = "dispatch_assert_queue_not$V2")]
     /// Generates an assertion if the current block is executing on the specified dispatch queue.
     pub fn dispatch_assert_queue_not(queue: dispatch_queue_t);
 
@@ -486,11 +474,11 @@ extern "C" {
     pub static _dispatch_source_type_data_replace: dispatch_source_type_s;
     pub static _dispatch_source_type_mach_send: dispatch_source_type_s;
     pub static _dispatch_source_type_memorypressure: dispatch_source_type_s;
-    #[cfg(any(target_os = "macos", target_os = "ios", target_os = "tvos"))]
+    #[cfg(target_vendor = "apple")]
     pub static _dispatch_source_type_proc: dispatch_source_type_s;
     pub static _dispatch_source_type_read: dispatch_source_type_s;
     pub static _dispatch_source_type_timer: dispatch_source_type_s;
-    #[cfg(any(target_os = "macos", target_os = "ios", target_os = "tvos"))]
+    #[cfg(target_vendor = "apple")]
     pub static _dispatch_source_type_vnode: dispatch_source_type_s;
     pub static _dispatch_source_type_write: dispatch_source_type_s;
     /// Creates a new dispatch source to monitor low-level system events.

crates/dispatch2/src/ffi.rs

@@ -118,7 +118,7 @@ impl Group {
     ///
     /// - Object shouldn't be released manually.
     pub const unsafe fn as_raw(&self) -> dispatch_group_t {
-        self.dispatch_object.as_raw()
+        unsafe { self.dispatch_object.as_raw() }
     }
 }
 

crates/dispatch2/src/group.rs

@@ -1,12 +1,4 @@
-#![allow(unused_unsafe, unreachable_patterns)]
-#![deny(
-    missing_docs,
-    clippy::undocumented_unsafe_blocks,
-    clippy::missing_safety_doc
-)]
-
-//!
-//! Apple Dispatch (Grand Central Dispatch)
+//! # Apple's Dispatch (Grand Central Dispatch)
 //!
 //! This crate allows interaction with the [Apple Dispatch](https://developer.apple.com/documentation/dispatch) library in a safe (``dispatch2`` module) and unsafe (``ffi`` module) way.
 //!
@@ -21,6 +13,12 @@
 //!     queue.exec_sync(|| println!("World"));
 //! }
 //! ```
+#![allow(unreachable_patterns)]
+#![warn(missing_docs)]
+#![warn(clippy::missing_errors_doc)]
+#![warn(clippy::missing_panics_doc)]
+#![warn(clippy::undocumented_unsafe_blocks)]
+#![warn(clippy::missing_safety_doc)]
 
 use self::ffi::dispatch_qos_class_t;
 

crates/dispatch2/src/lib.rs

@@ -171,7 +171,7 @@ impl Queue {
         }
     }
 
-    /// Return a system-defined global concurrent [Queue] with the priority derivated from [GlobalQueueIdentifier].
+    /// Return a system-defined global concurrent [Queue] with the priority derived from [GlobalQueueIdentifier].
     pub fn global_queue(identifier: GlobalQueueIdentifier) -> Self {
         let raw_identifier = identifier.to_identifier();
 
@@ -334,7 +334,7 @@ impl Queue {
     ///
     /// - Object shouldn't be released manually.
     pub const unsafe fn as_raw(&self) -> dispatch_queue_t {
-        self.dispatch_object.as_raw()
+        unsafe { self.dispatch_object.as_raw() }
     }
 }
 
@@ -388,7 +388,7 @@ impl WorkloopQueue {
     ///
     /// - Object shouldn't be released manually.
     pub const unsafe fn as_raw(&self) -> dispatch_workloop_t {
-        self.queue.as_raw() as dispatch_workloop_t
+        unsafe { self.queue.as_raw() as dispatch_workloop_t }
     }
 }
 

crates/dispatch2/src/queue.rs

@@ -35,7 +35,7 @@ impl Semaphore {
         Some(Semaphore { dispatch_object })
     }
 
-    /// Attempt to aquire the [Semaphore] and return a [SemaphoreGuard].
+    /// Attempt to acquire the [Semaphore] and return a [SemaphoreGuard].
     ///
     /// # Errors
     ///
@@ -72,7 +72,7 @@ impl Semaphore {
     ///
     /// - Object shouldn't be released manually.
     pub const unsafe fn as_raw(&self) -> dispatch_semaphore_t {
-        self.dispatch_object.as_raw()
+        unsafe { self.dispatch_object.as_raw() }
     }
 }