aboutsummaryrefslogtreecommitdiffhomepage
diff options
context:
space:
mode:
authorGravatar Paul Beusterien <paulbeusterien@google.com>2018-02-09 15:45:11 -0800
committerGravatar GitHub <noreply@github.com>2018-02-09 15:45:11 -0800
commit612d99c759741bd6384102f586aed1b3874cf95b (patch)
treece631ae10de2b9721450a50fe0477f74629afec2
parentbf45a15e4fcae11cde33bec23e7021d430b9af5f (diff)
Fix Core CLANG_WARN_OBJC_IMPLICIT_RETAIN_SELF warnings (#779)
-rw-r--r--Firebase/Core/FIRMutableDictionary.m14
-rw-r--r--Firebase/Core/FIRNetworkURLSession.m14
-rw-r--r--Firebase/Core/FIROptions.m2
3 files changed, 15 insertions, 15 deletions
diff --git a/Firebase/Core/FIRMutableDictionary.m b/Firebase/Core/FIRMutableDictionary.m
index 1d6ef3a..31941bc 100644
--- a/Firebase/Core/FIRMutableDictionary.m
+++ b/Firebase/Core/FIRMutableDictionary.m
@@ -37,7 +37,7 @@
- (NSString *)description {
__block NSString *description;
dispatch_sync(_queue, ^{
- description = _objects.description;
+ description = self->_objects.description;
});
return description;
}
@@ -45,33 +45,33 @@
- (id)objectForKey:(id)key {
__block id object;
dispatch_sync(_queue, ^{
- object = _objects[key];
+ object = self->_objects[key];
});
return object;
}
- (void)setObject:(id)object forKey:(id<NSCopying>)key {
dispatch_async(_queue, ^{
- _objects[key] = object;
+ self->_objects[key] = object;
});
}
- (void)removeObjectForKey:(id)key {
dispatch_async(_queue, ^{
- [_objects removeObjectForKey:key];
+ [self->_objects removeObjectForKey:key];
});
}
- (void)removeAllObjects {
dispatch_async(_queue, ^{
- [_objects removeAllObjects];
+ [self->_objects removeAllObjects];
});
}
- (NSUInteger)count {
__block NSUInteger count;
dispatch_sync(_queue, ^{
- count = _objects.count;
+ count = self->_objects.count;
});
return count;
}
@@ -89,7 +89,7 @@
- (NSDictionary *)dictionary {
__block NSDictionary *dictionary;
dispatch_sync(_queue, ^{
- dictionary = [_objects copy];
+ dictionary = [self->_objects copy];
});
return dictionary;
}
diff --git a/Firebase/Core/FIRNetworkURLSession.m b/Firebase/Core/FIRNetworkURLSession.m
index c3da674..6b5ce3a 100644
--- a/Firebase/Core/FIRNetworkURLSession.m
+++ b/Firebase/Core/FIRNetworkURLSession.m
@@ -314,11 +314,11 @@
if (allow) {
completionHandler(NSURLSessionAuthChallengeUseCredential, credential);
} else {
- [_loggerDelegate
+ [self->_loggerDelegate
firNetwork_logWithLevel:kFIRNetworkLogLevelDebug
messageCode:kFIRNetworkMessageCodeURLSession007
message:@"Cancelling authentication challenge for host. Host"
- context:_request.URL];
+ context:self->_request.URL];
completionHandler(NSURLSessionAuthChallengeCancelAuthenticationChallenge, nil);
}
};
@@ -344,10 +344,10 @@
}
if (trustError != errSecSuccess) {
- [_loggerDelegate firNetwork_logWithLevel:kFIRNetworkLogLevelError
- messageCode:kFIRNetworkMessageCodeURLSession008
- message:@"Cannot evaluate server trust. Error, host"
- contexts:@[ @(trustError), _request.URL ]];
+ [self->_loggerDelegate firNetwork_logWithLevel:kFIRNetworkLogLevelError
+ messageCode:kFIRNetworkMessageCodeURLSession008
+ message:@"Cannot evaluate server trust. Error, host"
+ contexts:@[ @(trustError), self->_request.URL ]];
shouldAllow = NO;
} else {
// Having a trust level "unspecified" by the user is the usual result, described at
@@ -651,7 +651,7 @@
if (handler) {
dispatch_async(dispatch_get_main_queue(), ^{
- handler(response, data, _sessionID, error);
+ handler(response, data, self->_sessionID, error);
});
}
}
diff --git a/Firebase/Core/FIROptions.m b/Firebase/Core/FIROptions.m
index fc92a55..2e490b6 100644
--- a/Firebase/Core/FIROptions.m
+++ b/Firebase/Core/FIROptions.m
@@ -367,7 +367,7 @@ static NSDictionary *sDefaultOptionsDictionary = nil;
}
tempAnalyticsOptions[key] = value;
}
- _analyticsOptionsDictionary = tempAnalyticsOptions;
+ self->_analyticsOptionsDictionary = tempAnalyticsOptions;
});
return _analyticsOptionsDictionary;
}