Initialise emit before constructing CBCentralManager (#14)

package.json

@@ -1,7 +1,7 @@
 {
   "name": "noble-mac",
   "description": "Noble (Node.js Bluetooth LE) bindings using CoreBluetooth API",
-  "version": "0.0.1",
+  "version": "0.0.2",
   "author": "[email protected]",
   "license": "MIT",
   "repository": {

src/ble_manager.h

@@ -14,14 +14,14 @@
 #include "callbacks.h"
 
 @interface BLEManager : NSObject <CBCentralManagerDelegate, CBPeripheralDelegate> {
-    @public Emit emit;
+    Emit emit;
     bool pendingRead;
 }
 @property (strong) CBCentralManager *centralManager;
 @property dispatch_queue_t dispatchQueue;
 @property NSMutableDictionary *peripherals;
 
-- (instancetype)init;
+- (instancetype)init: (const Napi::Value&) receiver with: (const Napi::Function&) callback;
 - (void)scan: (NSArray<NSString*> *)serviceUUIDs allowDuplicates: (BOOL)allowDuplicates;
 - (void)stopScan;
 - (BOOL)connect:(NSString*) uuid;

src/ble_manager.mm

@@ -11,9 +11,11 @@
 #include "objc_cpp.h"
 
 @implementation BLEManager
-- (instancetype)init   {
+- (instancetype)init: (const Napi::Value&) receiver with: (const Napi::Function&) callback {
     if (self = [super init]) {
         pendingRead = false;
+        // wrap cb before creating the CentralManager as it may call didUpdateState immediately
+        self->emit.Wrap(receiver, callback);
         self.dispatchQueue = dispatch_queue_create("CBqueue", 0);
         self.centralManager = [[CBCentralManager alloc] initWithDelegate:self queue:self.dispatchQueue];
         self.peripherals = [NSMutableDictionary dictionaryWithCapacity:10];

src/callbacks.h

@@ -1,9 +1,9 @@
-#pragma once
-
+#pragma once
+
 #include <napi.h>
 #include "peripheral.h"
-
-class ThreadSafeCallback;
+
+class ThreadSafeCallback;
 
 class Emit {
 public:

src/noble_mac.mm

@@ -47,8 +47,7 @@
 
 Napi::Value NobleMac::Init(const Napi::CallbackInfo& info) {
     Napi::Function emit = info.This().As<Napi::Object>().Get("emit").As<Napi::Function>();
-    manager = [[BLEManager alloc] init];
-    manager->emit.Wrap(info.This(), emit);
+    manager = [[BLEManager alloc] init:info.This() with:emit];
     return Napi::Value();
 }