Skip to content
This repository was archived by the owner on May 14, 2024. It is now read-only.

Commit 92d71e7

Browse files
committed
Merge commit '7f21f85633bfbe1867a47d2c6528f0bcd7af573d' into develop
# By Menno Pruijssers # Via Menno Pruijssers * commit '7f21f85633bfbe1867a47d2c6528f0bcd7af573d': Removed nill-checks Changed delegates from unsafe_unretained to weak Conflicts: SocketIOTransportXHR.h
2 parents 62fc2c6 + 7f21f85 commit 92d71e7

File tree

6 files changed

+21
-13
lines changed

6 files changed

+21
-13
lines changed

SocketIO.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,7 @@ typedef enum {
6767
NSString *_endpoint;
6868
NSDictionary *_params;
6969

70-
__unsafe_unretained id<SocketIODelegate> _delegate;
70+
__weak id<SocketIODelegate> _delegate;
7171

7272
NSObject <SocketIOTransport> *_transport;
7373

@@ -100,7 +100,7 @@ typedef enum {
100100
@property (nonatomic, readonly) NSTimeInterval heartbeatTimeout;
101101
@property (nonatomic) BOOL useSecure;
102102
@property (nonatomic, readonly) BOOL isConnected, isConnecting;
103-
@property (nonatomic, unsafe_unretained) id<SocketIODelegate> delegate;
103+
@property (nonatomic, weak) id<SocketIODelegate> delegate;
104104
@property (nonatomic) BOOL returnAllDataFromAck;
105105

106106
- (id) initWithDelegate:(id<SocketIODelegate>)delegate;

SocketIOTransport.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,6 +45,6 @@
4545
- (BOOL) isReady;
4646
- (void) send:(NSString *)request;
4747

48-
@property (nonatomic, unsafe_unretained) id <SocketIOTransportDelegate> delegate;
48+
@property (nonatomic, weak) id <SocketIOTransportDelegate> delegate;
4949

5050
@end

SocketIOTransportWebsocket.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,6 @@
3131
SRWebSocket *_webSocket;
3232
}
3333

34-
@property (nonatomic, unsafe_unretained) id <SocketIOTransportDelegate> delegate;
34+
@property (nonatomic, weak) id <SocketIOTransportDelegate> delegate;
3535

3636
@end

SocketIOTransportWebsocket.m

Lines changed: 11 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -99,7 +99,9 @@ - (void) send:(NSString*)request
9999

100100
- (void) webSocket:(SRWebSocket *)webSocket didReceiveMessage:(id)message
101101
{
102-
[delegate onData:message];
102+
if([delegate respondsToSelector:@selector(onData:)]) {
103+
[delegate onData:message];
104+
}
103105
}
104106

105107
- (void) webSocketDidOpen:(SRWebSocket *)webSocket
@@ -111,7 +113,9 @@ - (void) webSocket:(SRWebSocket *)webSocket didFailWithError:(NSError *)error
111113
{
112114
DEBUGLOG(@"Socket failed with error ... %@", [error localizedDescription]);
113115
// Assuming this resulted in a disconnect
114-
[delegate onDisconnect:error];
116+
if([delegate respondsToSelector:@selector(onDisconnect:)]) {
117+
[delegate onDisconnect:error];
118+
}
115119
}
116120

117121
- (void) webSocket:(SRWebSocket *)webSocket
@@ -120,9 +124,11 @@ - (void) webSocket:(SRWebSocket *)webSocket
120124
wasClean:(BOOL)wasClean
121125
{
122126
DEBUGLOG(@"Socket closed. %@", reason);
123-
[delegate onDisconnect:[NSError errorWithDomain:SocketIOError
124-
code:SocketIOWebSocketClosed
125-
userInfo:nil]];
127+
if([delegate respondsToSelector:@selector(onDisconnect:)]) {
128+
[delegate onDisconnect:[NSError errorWithDomain:SocketIOError
129+
code:SocketIOWebSocketClosed
130+
userInfo:nil]];
131+
}
126132
}
127133

128134
@end

SocketIOTransportXHR.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@
3333
BOOL _isClosed;
3434
}
3535

36-
@property (nonatomic, unsafe_unretained) id <SocketIOTransportDelegate> delegate;
36+
@property (nonatomic, weak) id <SocketIOTransportDelegate> delegate;
3737
@property (nonatomic) BOOL isClosed;
3838

3939
@end

SocketIOTransportXHR.m

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -247,9 +247,11 @@ - (void) connectionDidFinishLoading:(NSURLConnection *)connection
247247

248248
if (![message isEqualToString:@"1"]) {
249249
NSArray *messages = [self packetsFromPayload:message];
250-
[messages enumerateObjectsUsingBlock:^(NSString *message, NSUInteger idx, BOOL *stop) {
251-
[delegate onData:message];
252-
}];
250+
if([delegate respondsToSelector:@selector(onData:)]) {
251+
[messages enumerateObjectsUsingBlock:^(NSString *message, NSUInteger idx, BOOL *stop) {
252+
[delegate onData:message];
253+
}];
254+
}
253255
}
254256

255257
// remove current connection from pool

0 commit comments

Comments
 (0)