Add CoreBluetoothFeatures

Author: szymonlesisz

Cargo.toml

@@ -59,6 +59,7 @@ objc2-foundation = { version = "0.2.2", default-features = false, features = [
     "NSString",
     "NSUUID",
     "NSValue",
+    "NSProcessInfo",
 ] }
 objc2-core-bluetooth = { version = "0.2.2", default-features = false, features = [
     "std",

src/corebluetooth/internal.rs

@@ -31,7 +31,7 @@ use objc2_core_bluetooth::{
     CBCharacteristicProperties, CBCharacteristicWriteType, CBDescriptor, CBManager,
     CBManagerAuthorization, CBManagerState, CBPeripheral, CBPeripheralState, CBService, CBUUID,
 };
-use objc2_foundation::{NSArray, NSData, NSMutableDictionary, NSNumber};
+use objc2_foundation::{NSArray, NSData, NSMutableDictionary, NSNumber, NSProcessInfo};
 use std::{
     collections::{BTreeSet, HashMap, VecDeque},
     ffi::CString,
@@ -362,6 +362,12 @@ impl PeripheralInternal {
     }
 }
 
+/// Optional CoreBluetooth API capabilities that are available depending on the macOS version.
+struct CoreBluetoothFeatures {
+    /// `peripheral.canSendWriteWithoutResponse` property is available (since macOS 11.2.0)
+    can_send_write_without_response: bool,
+}
+
 // All of CoreBluetooth is basically async. It's all just waiting on delegate
 // events/callbacks. Therefore, we should be able to round up all of our wacky
 // ass mut *Object values, keep them in a single struct, in a single thread, and
@@ -376,6 +382,7 @@ struct CoreBluetoothInternal {
     // task::block this when sending even though it'll never actually block.
     event_sender: Sender<CoreBluetoothEvent>,
     message_receiver: Fuse<Receiver<CoreBluetoothMessage>>,
+    features: CoreBluetoothFeatures,
 }
 
 impl Debug for CoreBluetoothInternal {
@@ -474,6 +481,16 @@ pub enum CoreBluetoothEvent {
     },
 }
 
+fn get_features() -> CoreBluetoothFeatures {
+    let process_info = NSProcessInfo::processInfo();
+    let version = process_info.operatingSystemVersion();
+    let current = (version.majorVersion, version.minorVersion);
+
+    CoreBluetoothFeatures {
+        can_send_write_without_response: current >= (11, 2),
+    }
+}
+
 impl CoreBluetoothInternal {
     pub fn new(
         message_receiver: Receiver<CoreBluetoothMessage>,
@@ -499,6 +516,7 @@ impl CoreBluetoothInternal {
             event_sender,
             message_receiver: message_receiver.fuse(),
             delegate,
+            features: get_features(),
         }
     }
 
@@ -888,7 +906,9 @@ impl CoreBluetoothInternal {
                 {
                     trace!("Writing value! With kind {:?}", kind);
                     unsafe {
-                        if kind == WriteType::WithoutResponse {
+                        if kind == WriteType::WithoutResponse
+                            && self.features.can_send_write_without_response
+                        {
                             // probably better idea would be to wait for the result of peripheral.peripheralIsReadyToSendWriteWithoutResponse
                             let mut attempts = 0;
                             while !peripheral.peripheral.canSendWriteWithoutResponse()