From 612d99c759741bd6384102f586aed1b3874cf95b Mon Sep 17 00:00:00 2001 From: Paul Beusterien Date: Fri, 9 Feb 2018 15:45:11 -0800 Subject: Fix Core CLANG_WARN_OBJC_IMPLICIT_RETAIN_SELF warnings (#779) --- Firebase/Core/FIRMutableDictionary.m | 14 +++++++------- Firebase/Core/FIRNetworkURLSession.m | 14 +++++++------- Firebase/Core/FIROptions.m | 2 +- 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)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; } -- cgit v1.2.3