Add in default values to singletons, just to be explicit. Remove unnecessary synchronized from BranchJSONConfig

echo-branch · echo-branch · commit c93d8824d658 · 2024-04-04T10:30:07.000-07:00
diff --git a/Branch-TestBed/Branch-SDK-Tests/BNCEncodingUtilsTests.m b/Branch-TestBed/Branch-SDK-Tests/BNCEncodingUtilsTests.m
@@ -291,8 +291,8 @@ - (void)testDecodeQueryStringToDictionary {
 #pragma mark - Test Util methods
 
 - (NSString *)stringForDate:(NSDate *)date {
-    static NSDateFormatter *dateFormatter;
-    static dispatch_once_t onceToken;
+    static NSDateFormatter *dateFormatter = nil;
+    static dispatch_once_t onceToken = 0;
     
     dispatch_once(&onceToken, ^{
         dateFormatter = [[NSDateFormatter alloc] init];
diff --git a/Sources/BranchSDK/BNCAppGroupsData.m b/Sources/BranchSDK/BNCAppGroupsData.m
@@ -18,8 +18,8 @@ @interface BNCAppGroupsData()
 @implementation BNCAppGroupsData
 
 + (instancetype)shared {
-    static BNCAppGroupsData *appGroupsData;
-    static dispatch_once_t onceToken;
+    static BNCAppGroupsData *appGroupsData = nil;
+    static dispatch_once_t onceToken = 0;
     dispatch_once(&onceToken, ^{
         appGroupsData = [BNCAppGroupsData new];
     });
diff --git a/Sources/BranchSDK/BNCAppleReceipt.m b/Sources/BranchSDK/BNCAppleReceipt.m
@@ -23,8 +23,8 @@ @interface BNCAppleReceipt()
 @implementation BNCAppleReceipt
 
 + (BNCAppleReceipt *)sharedInstance {
-    static BNCAppleReceipt *singleton;
-    static dispatch_once_t onceToken;
+    static BNCAppleReceipt *singleton = nil;
+    static dispatch_once_t onceToken = 0;
     dispatch_once(&onceToken, ^{
         singleton = [BNCAppleReceipt new];
     });
diff --git a/Sources/BranchSDK/BNCCallbackMap.m b/Sources/BranchSDK/BNCCallbackMap.m
@@ -15,8 +15,8 @@ @interface BNCCallbackMap()
 @implementation BNCCallbackMap
 
 + (instancetype)shared {
-    static BNCCallbackMap *map;
-    static dispatch_once_t onceToken;
+    static BNCCallbackMap *map = nil;
+    static dispatch_once_t onceToken = 0;
     dispatch_once(&onceToken, ^{
         map = [BNCCallbackMap new];
     });
diff --git a/Sources/BranchSDK/BNCDeviceInfo.m b/Sources/BranchSDK/BNCDeviceInfo.m
@@ -34,7 +34,7 @@ @interface BNCDeviceInfo()
 @implementation BNCDeviceInfo
 
 + (BNCDeviceInfo *)getInstance {
-    static BNCDeviceInfo *bnc_deviceInfo = 0;
+    static BNCDeviceInfo *bnc_deviceInfo = nil;
     static dispatch_once_t onceToken = 0;
     dispatch_once(&onceToken, ^{
         bnc_deviceInfo = [BNCDeviceInfo new];
diff --git a/Sources/BranchSDK/BNCEncodingUtils.m b/Sources/BranchSDK/BNCEncodingUtils.m
@@ -129,9 +129,8 @@ + (NSString *)sha256Encode:(NSString *)input {
 #pragma mark - Param Encoding methods
 
 + (NSString *)iso8601StringFromDate:(NSDate *)date {
-    static NSDateFormatter *dateFormatter;
-    static dispatch_once_t onceToken;
-    
+    static NSDateFormatter *dateFormatter = nil;
+    static dispatch_once_t onceToken = 0;
     dispatch_once(&onceToken, ^{
         dateFormatter = [[NSDateFormatter alloc] init];
         [dateFormatter setLocale:[NSLocale localeWithLocaleIdentifier:@"en_US_POSIX"]]; // POSIX to avoid weird issues
diff --git a/Sources/BranchSDK/BNCEventUtils.m b/Sources/BranchSDK/BNCEventUtils.m
@@ -15,8 +15,8 @@ @interface BNCEventUtils()
 @implementation BNCEventUtils
 
 + (instancetype)shared {
-    static BNCEventUtils *set;
-    static dispatch_once_t onceToken;
+    static BNCEventUtils *set = nil;
+    static dispatch_once_t onceToken = 0;
     dispatch_once(&onceToken, ^{
         set = [BNCEventUtils new];
     });
diff --git a/Sources/BranchSDK/BNCPartnerParameters.m b/Sources/BranchSDK/BNCPartnerParameters.m
@@ -16,8 +16,8 @@ @interface BNCPartnerParameters()
 @implementation BNCPartnerParameters
 
 + (instancetype)shared {
-    static BNCPartnerParameters *partnerParameters;
-    static dispatch_once_t onceToken;
+    static BNCPartnerParameters *partnerParameters = nil;
+    static dispatch_once_t onceToken = 0;
     dispatch_once(&onceToken, ^{
         partnerParameters = [BNCPartnerParameters new];
     });
diff --git a/Sources/BranchSDK/BNCPasteboard.m b/Sources/BranchSDK/BNCPasteboard.m
@@ -13,8 +13,8 @@
 @implementation BNCPasteboard
 
 + (BNCPasteboard *)sharedInstance {
-    static BNCPasteboard *pasteboard;
-    static dispatch_once_t onceToken;
+    static BNCPasteboard *pasteboard = nil;
+    static dispatch_once_t onceToken = 0;
     dispatch_once(&onceToken, ^{
         pasteboard = [BNCPasteboard new];
     });
diff --git a/Sources/BranchSDK/BNCPreferenceHelper.m b/Sources/BranchSDK/BNCPreferenceHelper.m
@@ -122,8 +122,8 @@ @implementation BNCPreferenceHelper
     adUserDataUsageConsent = _adUserDataUsageConsent;
 
 + (BNCPreferenceHelper *)sharedInstance {
-    static BNCPreferenceHelper *preferenceHelper;
-    static dispatch_once_t onceToken;
+    static BNCPreferenceHelper *preferenceHelper = nil;
+    static dispatch_once_t onceToken = 0;
     
     dispatch_once(&onceToken, ^{
         preferenceHelper = [[BNCPreferenceHelper alloc] init];
diff --git a/Sources/BranchSDK/BNCQRCodeCache.m b/Sources/BranchSDK/BNCQRCodeCache.m
@@ -16,8 +16,8 @@ @implementation BNCQRCodeCache
 
 //Can only hold one QR code in cache. Just used to debounce.
 + (BNCQRCodeCache *) sharedInstance {
-    static BNCQRCodeCache *singleton;
-    static dispatch_once_t onceToken;
+    static BNCQRCodeCache *singleton = nil;
+    static dispatch_once_t onceToken = 0;
     dispatch_once(&onceToken, ^{
         singleton = [BNCQRCodeCache new];
     });
diff --git a/Sources/BranchSDK/BNCReachability.m b/Sources/BranchSDK/BNCReachability.m
@@ -28,8 +28,8 @@ @interface BNCReachability()
 @implementation BNCReachability
 
 + (BNCReachability *)shared {
-    static BNCReachability *reachability;
-    static dispatch_once_t onceToken;
+    static BNCReachability *reachability = nil;
+    static dispatch_once_t onceToken = 0;
     dispatch_once(&onceToken, ^{
         reachability = [BNCReachability new];
     });
diff --git a/Sources/BranchSDK/BNCSKAdNetwork.m b/Sources/BranchSDK/BNCSKAdNetwork.m
@@ -27,8 +27,8 @@ @interface BNCSKAdNetwork()
 @implementation BNCSKAdNetwork
 
 + (BNCSKAdNetwork *)sharedInstance {
-    static BNCSKAdNetwork *singleton;
-    static dispatch_once_t onceToken;
+    static BNCSKAdNetwork *singleton = nil;
+    static dispatch_once_t onceToken = 0;
     dispatch_once(&onceToken, ^{
         singleton = [[BNCSKAdNetwork alloc] init];
     });
diff --git a/Sources/BranchSDK/BNCServerAPI.m b/Sources/BranchSDK/BNCServerAPI.m
@@ -14,8 +14,8 @@
 @implementation BNCServerAPI
 
 + (BNCServerAPI *)sharedInstance {
-    static BNCServerAPI *serverAPI;
-    static dispatch_once_t onceToken;
+    static BNCServerAPI *serverAPI = nil;
+    static dispatch_once_t onceToken = 0;
     
     dispatch_once(&onceToken, ^{
         serverAPI = [[BNCServerAPI alloc] init];
diff --git a/Sources/BranchSDK/BNCServerRequestQueue.m b/Sources/BranchSDK/BNCServerRequestQueue.m
@@ -340,7 +340,7 @@ - (id)unarchiveObjectFromData:(NSData *)data {
 // only replay analytics requests, the others are time sensitive
 + (NSSet<Class> *)replayableRequestClasses {
     static NSSet<Class> *requestClasses = nil;
-    static dispatch_once_t onceToken;
+    static dispatch_once_t onceToken = 0;
     dispatch_once(&onceToken, ^ {
         NSArray *tmp = @[
             [BranchOpenRequest class],
@@ -356,7 +356,7 @@ - (id)unarchiveObjectFromData:(NSData *)data {
 // encodable classes also includes NSArray and NSData
 + (NSSet<Class> *)encodableClasses {
     static NSSet<Class> *classes = nil;
-    static dispatch_once_t onceToken;
+    static dispatch_once_t onceToken = 0;
     dispatch_once(&onceToken, ^ {
         NSMutableArray *tmp = [NSMutableArray new];
         [tmp addObject:[NSArray class]]; // root object
@@ -373,7 +373,7 @@ - (id)unarchiveObjectFromData:(NSData *)data {
 
 + (NSURL * _Nonnull) URLForQueueFile {
     static NSURL *URL = nil;
-    static dispatch_once_t onceToken;
+    static dispatch_once_t onceToken = 0;
     dispatch_once(&onceToken, ^ {
         URL = BNCURLForBranchDirectory();
         URL = [URL URLByAppendingPathComponent:BRANCH_QUEUE_FILE isDirectory:NO];
@@ -383,7 +383,7 @@ + (NSURL * _Nonnull) URLForQueueFile {
 
 + (instancetype)getInstance {
     static BNCServerRequestQueue *sharedQueue = nil;
-    static dispatch_once_t onceToken;
+    static dispatch_once_t onceToken = 0;
     dispatch_once(&onceToken, ^ {
         sharedQueue = [[BNCServerRequestQueue alloc] init];
         [sharedQueue retrieve];
diff --git a/Sources/BranchSDK/BNCUserAgentCollector.m b/Sources/BranchSDK/BNCUserAgentCollector.m
@@ -26,8 +26,8 @@ @interface BNCUserAgentCollector()
 @implementation BNCUserAgentCollector
 
 + (BNCUserAgentCollector *)instance {
-    static BNCUserAgentCollector *collector;
-    static dispatch_once_t onceToken;
+    static BNCUserAgentCollector *collector = nil;
+    static dispatch_once_t onceToken = 0;
     dispatch_once(&onceToken, ^{
         collector = [BNCUserAgentCollector new];
     });
diff --git a/Sources/BranchSDK/BranchContentDiscoverer.m b/Sources/BranchSDK/BranchContentDiscoverer.m
@@ -26,8 +26,8 @@ @interface BranchContentDiscoverer ()
 @implementation BranchContentDiscoverer
 
 + (BranchContentDiscoverer *)getInstance {
-    static BranchContentDiscoverer *sharedInstance;
-    static dispatch_once_t onceToken;
+    static BranchContentDiscoverer *sharedInstance = nil;
+    static dispatch_once_t onceToken = 0;
     dispatch_once(&onceToken, ^{
         sharedInstance = [BranchContentDiscoverer new];
     });
diff --git a/Sources/BranchSDK/BranchContentDiscoveryManifest.m b/Sources/BranchSDK/BranchContentDiscoveryManifest.m
@@ -34,8 +34,8 @@ - (instancetype)init {
 }
 
 + (BranchContentDiscoveryManifest *)getInstance {
-    static BranchContentDiscoveryManifest *sharedInstance;
-    static dispatch_once_t onceToken;
+    static BranchContentDiscoveryManifest *sharedInstance = nil;
+    static dispatch_once_t onceToken = 0;
     dispatch_once(&onceToken, ^{
         sharedInstance = [BranchContentDiscoveryManifest new];
     });
diff --git a/Sources/BranchSDK/BranchJsonConfig.m b/Sources/BranchSDK/BranchJsonConfig.m
@@ -28,16 +28,13 @@ @interface BranchJsonConfig()
 
 @implementation BranchJsonConfig
 
-+ (BranchJsonConfig * _Nonnull)instance
-{
-    @synchronized(self) {
-        static BranchJsonConfig *_instance;
-        static dispatch_once_t once = 0;
-        dispatch_once(&once, ^{
-            _instance = [[BranchJsonConfig alloc] init];
-        });
-        return _instance;
-    }
++ (BranchJsonConfig * _Nonnull)instance {
+    static BranchJsonConfig *instance = nil;
+    static dispatch_once_t once = 0;
+    dispatch_once(&once, ^{
+        instance = [BranchJsonConfig new];
+    });
+    return instance;
 }
 
 - (instancetype)init
diff --git a/Sources/BranchSDK/BranchLogger.m b/Sources/BranchSDK/BranchLogger.m
@@ -31,7 +31,7 @@ - (instancetype)init {
 
 + (instancetype)shared {
     static BranchLogger *sharedInstance = nil;
-    static dispatch_once_t onceToken;
+    static dispatch_once_t onceToken = 0;
     dispatch_once(&onceToken, ^{
         sharedInstance = [[BranchLogger alloc] init];
         sharedInstance.loggingEnabled = NO;
diff --git a/Sources/BranchSDK/BranchPluginSupport.m b/Sources/BranchSDK/BranchPluginSupport.m
@@ -16,8 +16,8 @@
 @implementation BranchPluginSupport
 
 + (BranchPluginSupport *)instance {
-    static BranchPluginSupport *pluginSupport;
-    static dispatch_once_t onceToken;
+    static BranchPluginSupport *pluginSupport = nil;
+    static dispatch_once_t onceToken = 0;
     dispatch_once(&onceToken, ^{
         pluginSupport = [BranchPluginSupport new];
     });
diff --git a/Sources/BranchSDK/BranchScene.m b/Sources/BranchSDK/BranchScene.m
@@ -13,8 +13,8 @@
 @implementation BranchScene
 
 + (BranchScene *)shared NS_EXTENSION_UNAVAILABLE("BranchScene does not support Extensions") {
-    static BranchScene *bscene;
-    static dispatch_once_t onceToken;
+    static BranchScene *bscene = nil;
+    static dispatch_once_t onceToken = 0;
     dispatch_once(&onceToken, ^{
         bscene = [BranchScene new];
     });
diff --git a/Sources/BranchSDK/NSError+Branch.m b/Sources/BranchSDK/NSError+Branch.m
@@ -22,8 +22,8 @@ + (NSString *)bncErrorDomain {
 
 // Legacy error messages
 + (NSString *)messageForCode:(BNCErrorCode)code {
-    static NSMutableDictionary<NSNumber *, NSString *> *messages;
-    static dispatch_once_t onceToken;
+    static NSMutableDictionary<NSNumber *, NSString *> *messages = nil;
+    static dispatch_once_t onceToken = 0;
     dispatch_once(&onceToken, ^{
         messages = [NSMutableDictionary<NSNumber *, NSString *> new];
         [messages setObject:@"The Branch user session has not been initialized." forKey:@(BNCInitError)];
