Skip to content

Commit cee34dd

Browse files
fix: cleanup type names
1 parent 06efacc commit cee34dd

File tree

3 files changed

+15
-20
lines changed

3 files changed

+15
-20
lines changed

DevCycle/DevCycleClient.swift

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -391,11 +391,11 @@ public class DevCycleClient {
391391
return self.config?.userConfig?.variables ?? [:]
392392
}
393393

394-
public func subscribe(_ handler: SubscribeHandlers) {
394+
public func subscribe(_ handler: DevCycleEventHandlers) {
395395
self.eventEmitter.subscribe(handler)
396396
}
397397

398-
public func unsubscribe(_ handler: SubscribeHandlers) {
398+
public func unsubscribe(_ handler: DevCycleEventHandlers) {
399399
self.eventEmitter.unsubscribe(handler)
400400
}
401401

DevCycle/Utils/EventEmitter.swift

Lines changed: 10 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -8,39 +8,36 @@ public typealias EventHandler = (Any?) -> Void
88
public typealias SubscribeErrorHandler = (Error) -> Void
99
public typealias SubscribeInitializedHandler = (Bool) -> Void
1010

11-
public struct SubscribeError {
11+
public struct ErrorHandler {
1212
// Had to add this index field, because I can't make this struct Equitable to do a equality comparison
1313
// because you can't compare anon function pointers... so it was either this or generate a UUID to compare.
1414
fileprivate var index: Int?
1515
var handler: SubscribeErrorHandler
16-
let key = "error"
17-
1816

1917
public init(_ handler: @escaping SubscribeErrorHandler) {
2018
self.handler = handler
2119
}
2220
}
2321

24-
public struct SubscribeInitialized {
22+
public struct InitializedHandler {
2523
fileprivate var index: Int?
2624
var handler: SubscribeInitializedHandler
27-
let key = "initialized"
2825

2926
public init(_ handler: @escaping SubscribeInitializedHandler) {
3027
self.handler = handler
3128
}
3229
}
3330

34-
public enum SubscribeHandlers {
35-
case error(SubscribeError)
36-
case initialized(SubscribeInitialized)
31+
public enum DevCycleEventHandlers {
32+
case error(ErrorHandler)
33+
case initialized(InitializedHandler)
3734
}
3835

3936
class EventEmitter {
40-
var errorHandlers: [SubscribeError] = []
41-
var initHandlers: [SubscribeInitialized] = []
37+
var errorHandlers: [ErrorHandler] = []
38+
var initHandlers: [InitializedHandler] = []
4239

43-
func subscribe(_ handler: SubscribeHandlers) {
40+
func subscribe(_ handler: DevCycleEventHandlers) {
4441
switch handler {
4542
case .error(var errorHandler):
4643
self.errorHandlers.append(errorHandler)
@@ -53,7 +50,7 @@ class EventEmitter {
5350
}
5451
}
5552

56-
func unsubscribe(_ handler: SubscribeHandlers) {
53+
func unsubscribe(_ handler: DevCycleEventHandlers) {
5754
switch handler {
5855
case .error(let errorHandler):
5956
if let index = errorHandler.index {
@@ -69,9 +66,7 @@ class EventEmitter {
6966
}
7067

7168
func emitError(_ err: Error) {
72-
self.errorHandlers.forEach { errSub in
73-
errSub.handler(err)
74-
}
69+
self.errorHandlers.forEach { errHandler in errHandler.handler(err) }
7570
}
7671

7772
func emitInitialized(_ success: Bool) {

Examples/DevCycle-iOS-Example-App-Swift/DevCycle-iOS-Example-App-Swift/DevCycleManager.swift

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -32,11 +32,11 @@ class DevCycleManager {
3232
self.client = client
3333

3434

35-
client.subscribe(SubscribeHandlers.error(SubscribeError { error in
35+
client.subscribe(DevCycleEventHandlers.error(ErrorHandler { error in
3636
print("DevCycle Error: \(error.localizedDescription)")
3737
}))
38-
client.subscribe(SubscribeHandlers.initialized(SubscribeInitialized({ success in
38+
client.subscribe(DevCycleEventHandlers.initialized(InitializedHandler { success in
3939
print("DevCycle Initialized: \(success) from subscription")
40-
})))
40+
}))
4141
}
4242
}

0 commit comments

Comments
 (0)