Skip to content

Commit

Permalink
Change return type of init methods to instancetype instead of id
Browse files Browse the repository at this point in the history
  • Loading branch information
Benjamin Herzog committed Jun 23, 2016
1 parent 27db3ac commit 8ec6a28
Show file tree
Hide file tree
Showing 47 changed files with 62 additions and 60 deletions.
2 changes: 1 addition & 1 deletion ARAnalyticalProvider.h
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ extern NSString *const ARAnalyticalProviderNewPageViewEventScreenPropertyKey;
@interface ARAnalyticalProvider : NSObject

/// Init
- (id)initWithIdentifier:(NSString *)identifier;
- (instancetype)initWithIdentifier:(NSString *)identifier;

/// Set a per user property
- (void)setUserProperty:(NSString *)property toValue:(id)value;
Expand Down
2 changes: 1 addition & 1 deletion ARAnalyticalProvider.m
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ @interface ARAnalyticalProvider () {

@implementation ARAnalyticalProvider

- (id)initWithIdentifier:(NSString *)identifier {
- (instancetype)initWithIdentifier:(NSString *)identifier {
return [super init];
}

Expand Down
2 changes: 1 addition & 1 deletion Providers/AdjustProvider.m
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@

@implementation AdjustProvider

- (id)initWithIdentifier:(NSString *)identifier {
- (instancetype)initWithIdentifier:(NSString *)identifier {
return [self initWithIdentifier:identifier andConfigurationDelegate:nil];
}

Expand Down
4 changes: 2 additions & 2 deletions Providers/AdobeProvider.h
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ extern NSString const *AdobeProviderCallsTrackStateForPageViewsWithProperties;

@interface AdobeProvider : ARAnalyticalProvider

- (id)initWithData:(NSDictionary *)additionalData;
- (id)initWithData:(NSDictionary *)additionalData settings:(NSDictionary*)settings;
- (instancetype)initWithData:(NSDictionary *)additionalData;
- (instancetype)initWithData:(NSDictionary *)additionalData settings:(NSDictionary*)settings;

@end
6 changes: 3 additions & 3 deletions Providers/AdobeProvider.m
Original file line number Diff line number Diff line change
Expand Up @@ -18,11 +18,11 @@ @interface AdobeProvider ()
@implementation AdobeProvider
#ifdef AR_ADOBE_EXISTS

- (id)initWithData:(NSDictionary *)additionalData {
- (instancetype)initWithData:(NSDictionary *)additionalData {
return [self initWithData:additionalData settings:nil];
}

- (id)initWithData:(NSDictionary *)additionalData settings:(NSDictionary*)settings {
- (instancetype)initWithData:(NSDictionary *)additionalData settings:(NSDictionary*)settings {
NSAssert([ADBMobile class], @"Adobe is not included");
if(additionalData) {
[ADBMobile collectLifecycleDataWithAdditionalData:additionalData];
Expand All @@ -37,7 +37,7 @@ - (id)initWithData:(NSDictionary *)additionalData settings:(NSDictionary*)settin
return self;
}

- (id)initWithIdentifier:(NSString *)identifier
- (instancetype)initWithIdentifier:(NSString *)identifier
{
return [self initWithData:nil];
}
Expand Down
2 changes: 1 addition & 1 deletion Providers/AmplitudeProvider.m
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
@implementation AmplitudeProvider
#ifdef AR_AMPLITUDE_EXISTS

-(id)initWithIdentifier:(NSString *)identifier {
-(instancetype)initWithIdentifier:(NSString *)identifier {
NSAssert([Amplitude class], @"Amplitude is not included");
[[Amplitude instance] initializeApiKey:identifier];

Expand Down
2 changes: 1 addition & 1 deletion Providers/AppsFlyerProvider.m
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@
@implementation AppsFlyerProvider
#ifdef AR_APPSFLYER_EXISTS

- (id)initWithIdentifier:(NSString *)identifier {
- (instancetype)initWithIdentifier:(NSString *)identifier {
NSLog(@"Use -[AppsFlyerProvider initWithAppID:devKey:] instead of %s", __PRETTY_FUNCTION__);
[self doesNotRecognizeSelector:_cmd];
return nil;
Expand Down
2 changes: 1 addition & 1 deletion Providers/AppseeProvider.m
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
@implementation AppseeProvider
#ifdef AR_APPSEE_EXISTS

-(id)initWithIdentifier:(NSString *)identifier {
-(instancetype)initWithIdentifier:(NSString *)identifier {
NSAssert([Appsee class], @"Appsee is not included");
[Appsee start:identifier];

Expand Down
2 changes: 1 addition & 1 deletion Providers/BranchProvider.h
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,6 @@

@interface BranchProvider : ARAnalyticalProvider

- (id)initWithAPIKey:(NSString *)key;
- (instancetype)initWithAPIKey:(NSString *)key;

@end
2 changes: 1 addition & 1 deletion Providers/BranchProvider.m
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
@implementation BranchProvider

#ifdef AR_BRANCH_EXISTS
- (id)initWithAPIKey:(NSString *)key {
- (instancetype)initWithAPIKey:(NSString *)key {
NSAssert([Branch class], @"Branch is not included");
[Branch getInstance:key];
return [super init];
Expand Down
2 changes: 1 addition & 1 deletion Providers/BugsnagProvider.m
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
@implementation BugsnagProvider
#ifdef AR_BUGSNAP_EXISTS

- (id)initWithIdentifier:(NSString *)identifier {
- (instancetype)initWithIdentifier:(NSString *)identifier {
NSAssert([Bugsnag class], @"Bugsnag is not included");
[Bugsnag startBugsnagWithApiKey:identifier];

Expand Down
2 changes: 1 addition & 1 deletion Providers/ChartbeatProvider.m
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ @interface ChartbeatProvider ()
@implementation ChartbeatProvider
#ifdef AR_CHARTBEAT_EXISTS

- (id)initWithIdentifier:(NSString *)identifier
- (instancetype)initWithIdentifier:(NSString *)identifier
{
[[CBTracker sharedTracker] startTrackerWithAccountID:identifier.intValue];
return self;
Expand Down
4 changes: 3 additions & 1 deletion Providers/CountlyProvider.h
Original file line number Diff line number Diff line change
@@ -1,5 +1,7 @@
#import "ARAnalyticalProvider.h"

@interface CountlyProvider : ARAnalyticalProvider
- (id)initWithAppKey:(NSString *)appKey andHost:(NSString *)host;

- (instancetype)initWithAppKey:(NSString *)appKey andHost:(NSString *)host;

@end
2 changes: 1 addition & 1 deletion Providers/CountlyProvider.m
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@

@implementation CountlyProvider

- (id)initWithAppKey:(NSString *)appKey andHost:(NSString *)host {
- (instancetype)initWithAppKey:(NSString *)appKey andHost:(NSString *)host {
#ifdef AR_COUNTLY_EXISTS
NSAssert([Countly class], @"Countly is not included");
if (host) {
Expand Down
2 changes: 1 addition & 1 deletion Providers/CrashlyticsProvider.m
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
@implementation CrashlyticsProvider
#ifdef AR_CRASHLYTICS_EXISTS

- (id)initWithIdentifier:(NSString *)identifier {
- (instancetype)initWithIdentifier:(NSString *)identifier {
NSAssert([Crashlytics class], @"Crashlytics is not included");
NSAssert([[Crashlytics class] respondsToSelector:@selector(version)], @"Crashlytics library not installed correctly.");
[Crashlytics startWithAPIKey:identifier];
Expand Down
2 changes: 1 addition & 1 deletion Providers/CrittercismProvider.m
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
@implementation CrittercismProvider
#ifdef AR_CRITTERCISM_EXISTS

- (id)initWithIdentifier:(NSString *)identifier {
- (instancetype)initWithIdentifier:(NSString *)identifier {
NSAssert([Crittercism class], @"Crittercism is not included");
[Crittercism enableWithAppID:identifier];

Expand Down
2 changes: 1 addition & 1 deletion Providers/FabricProvider.h
Original file line number Diff line number Diff line change
Expand Up @@ -16,5 +16,5 @@ OBJC_EXTERN void CLSLog(NSString *format, ...) NS_FORMAT_FUNCTION(1,2);


@interface FabricProvider : ARAnalyticalProvider
- (id)initWithKits:(NSArray *)kits;
- (instancetype)initWithKits:(NSArray *)kits;
@end
2 changes: 1 addition & 1 deletion Providers/FabricProvider.m
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@ + (void)logCustomEventWithName:(NSString *)eventName
@implementation FabricProvider
#ifdef AR_FABRIC_EXISTS

- (id)initWithKits:(NSArray *)kits {
- (instancetype)initWithKits:(NSArray *)kits {

NSAssert([Fabric class], @"Fabric is not included");
NSAssert([[Fabric class] respondsToSelector:@selector(sharedSDK)], @"Fabric library not installed correctly.");
Expand Down
2 changes: 1 addition & 1 deletion Providers/FirebaseProvider.m
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
@implementation FirebaseProvider
#ifdef AR_FIREBASE_EXISTS

- (id)initWithIdentifier:(NSString *)identifier {
- (instancetype)initWithIdentifier:(NSString *)identifier {
NSAssert([FIRApp class], @"Firebase SDK is not included");
NSAssert([FIRAnalytics class], @"Firebase SDK is not included");

Expand Down
2 changes: 1 addition & 1 deletion Providers/FlurryProvider.m
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
@implementation FlurryProvider
#ifdef AR_FLURRY_EXISTS

- (id)initWithIdentifier:(NSString *)identifier {
- (instancetype)initWithIdentifier:(NSString *)identifier {
NSAssert([Flurry class], @"Flurry is not included");
[Flurry startSession:identifier];

Expand Down
2 changes: 1 addition & 1 deletion Providers/GoogleAnalyticsProvider.m
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ - (void) dispatchGA;
@implementation GoogleAnalyticsProvider
#ifdef AR_GOOGLEANALYTICS_EXISTS

- (id)initWithIdentifier:(NSString *)identifier {
- (instancetype)initWithIdentifier:(NSString *)identifier {
NSAssert([GAI class], @"Google Analytics SDK is not included");

if ((self = [super init])) {
Expand Down
2 changes: 1 addition & 1 deletion Providers/HeapAnalyticsProvider.m
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ @implementation HeapAnalyticsProvider

#ifdef AR_HEAPANALYTICS_EXISTS

-(id)initWithIdentifier:(id)identifier {
-(instancetype)initWithIdentifier:(id)identifier {
NSAssert([Heap class], @"Heap is not included");
[Heap setAppId:identifier];

Expand Down
2 changes: 1 addition & 1 deletion Providers/HelpshiftProvider.h
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@

@interface HelpshiftProvider : ARAnalyticalProvider

- (id)initWithAppID:(NSString *)appID domainName:(NSString *)domainName apiKey:(NSString *)apiKey;
- (instancetype)initWithAppID:(NSString *)appID domainName:(NSString *)domainName apiKey:(NSString *)apiKey;

@end

2 changes: 1 addition & 1 deletion Providers/HelpshiftProvider.m
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
@implementation HelpshiftProvider
#ifdef AR_HELPSHIFT_EXISTS

- (id)initWithAppID:(NSString *)appID domainName:(NSString *)domainName apiKey:(NSString *)apiKey {
- (instancetype)initWithAppID:(NSString *)appID domainName:(NSString *)domainName apiKey:(NSString *)apiKey {
NSAssert([Helpshift class], @"Helpshift is not included");

[Helpshift installForApiKey:apiKey domainName:domainName appID:appID withOptions:nil];
Expand Down
2 changes: 1 addition & 1 deletion Providers/HockeyAppOSXProvider.m
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ @interface HockeyAppOSXProvider () <BITHockeyManagerDelegate, BITCrashManagerDel

@implementation HockeyAppOSXProvider

-(id)initWithIdentifier:(NSString *)identifier {
-(instancetype)initWithIdentifier:(NSString *)identifier {
self = [super init];
if (!self) return nil;

Expand Down
2 changes: 1 addition & 1 deletion Providers/HockeyAppProvider.h
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
#import "ARAnalyticalProvider.h"

@interface HockeyAppProvider : ARAnalyticalProvider
-(id)initWithBetaIdentifier:(NSString *)betaIdentifier liveIdentifier:(NSString *)liveIdentfier;
-(instancetype)initWithBetaIdentifier:(NSString *)betaIdentifier liveIdentifier:(NSString *)liveIdentfier;
@end
4 changes: 2 additions & 2 deletions Providers/HockeyAppProvider.m
Original file line number Diff line number Diff line change
Expand Up @@ -43,11 +43,11 @@ @interface HockeyAppProvider () <BITHockeyManagerDelegate, BITUpdateManagerDeleg

@implementation HockeyAppProvider

-(id)initWithIdentifier:(NSString *)identifier {
-(instancetype)initWithIdentifier:(NSString *)identifier {
return [self initWithBetaIdentifier:identifier liveIdentifier:nil];
}

-(id)initWithBetaIdentifier:(NSString *)betaIdentifier liveIdentifier:(NSString *)liveIdentfier {
-(instancetype)initWithBetaIdentifier:(NSString *)betaIdentifier liveIdentifier:(NSString *)liveIdentfier {
self = [super init];
if (!self) return nil;

Expand Down
2 changes: 1 addition & 1 deletion Providers/InstallTrackerProvider.m
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
@implementation InstallTrackerProvider
#ifdef AR_INSTALLTRACKER_EXISTS

- (id)initWithIdentifier:(NSString *)identifier {
- (instancetype)initWithIdentifier:(NSString *)identifier {
NSAssert([InstallTracker class], @"InstallTracker is not included");
[InstallTracker setApplicationID:identifier];

Expand Down
2 changes: 1 addition & 1 deletion Providers/KISSMetricsProvider.m
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
@implementation KISSMetricsProvider
#ifdef AR_KISSMETRICS_EXISTS

- (id)initWithIdentifier:(NSString *)identifier {
- (instancetype)initWithIdentifier:(NSString *)identifier {
NSAssert([KISSMetricsAPI class], @"KISSMetrics is not included");
[KISSMetricsAPI sharedAPIWithKey:identifier];

Expand Down
2 changes: 1 addition & 1 deletion Providers/LaunchKitProvider.m
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
@implementation LaunchKitProvider
#ifdef AR_LAUNCHKIT_EXISTS

-(id)initWithIdentifier:(NSString *)identifier {
-(instancetype)initWithIdentifier:(NSString *)identifier {
NSAssert([LaunchKit class], @"LaunchKit is not included");
[LaunchKit launchWithToken:identifier];

Expand Down
2 changes: 1 addition & 1 deletion Providers/LeanplumProvider.h
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

@interface LeanplumProvider : ARAnalyticalProvider

- (id)initWithIdentifier:(NSString *)identifier NS_UNAVAILABLE;
- (instancetype)initWithIdentifier:(NSString *)identifier NS_UNAVAILABLE;

/**
* Initializes an instance of LeanplumProvider with the given parameters.
Expand Down
2 changes: 1 addition & 1 deletion Providers/LocalyticsProvider.m
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ - (void) stopLocalytics;
@implementation LocalyticsProvider
#ifdef AR_LOCALYTICS_EXISTS

- (id)initWithIdentifier:(NSString *)identifier {
- (instancetype)initWithIdentifier:(NSString *)identifier {
NSAssert([Localytics class], @"Localytics is not included");

if( ( self = [super init] ) ) {
Expand Down
2 changes: 1 addition & 1 deletion Providers/MixpanelProvider.h
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
#import "ARAnalyticalProvider.h"

@interface MixpanelProvider : ARAnalyticalProvider
- (id)initWithIdentifier:(NSString *)identifier andHost:(NSString *)host;
- (instancetype)initWithIdentifier:(NSString *)identifier andHost:(NSString *)host;
- (void)createAlias:(NSString *)alias;
- (void)registerSuperProperties:(NSDictionary *)properties;
- (void)addPushDeviceToken:(NSData *)deviceToken;
Expand Down
4 changes: 2 additions & 2 deletions Providers/MixpanelProvider.m
Original file line number Diff line number Diff line change
Expand Up @@ -10,11 +10,11 @@ @interface MixpanelProvider()

@implementation MixpanelProvider

- (id)initWithIdentifier:(NSString *)identifier {
- (instancetype)initWithIdentifier:(NSString *)identifier {
return [self initWithIdentifier:identifier andHost:nil];
}

- (id)initWithIdentifier:(NSString *)identifier andHost:(NSString *)host {
- (instancetype)initWithIdentifier:(NSString *)identifier andHost:(NSString *)host {
#ifdef AR_MIXPANEL_EXISTS

NSAssert([Mixpanel class], @"Mixpanel is not included");
Expand Down
2 changes: 1 addition & 1 deletion Providers/NewRelicProvider.m
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@

@implementation NewRelicProvider

- (id)initWithIdentifier:(NSString *)identifier {
- (instancetype)initWithIdentifier:(NSString *)identifier {
NSAssert([NewRelic class], @"NewRelic is not included");
[NewRelic startWithApplicationToken:identifier];

Expand Down
2 changes: 1 addition & 1 deletion Providers/SegmentioProvider.h
Original file line number Diff line number Diff line change
Expand Up @@ -6,5 +6,5 @@
#import "ARAnalyticalProvider.h"

@interface SegmentioProvider : ARAnalyticalProvider
- (id)initWithIdentifier:(NSString *)identifier integrations:(NSArray *)integrations;
- (instancetype)initWithIdentifier:(NSString *)identifier integrations:(NSArray *)integrations;
@end
4 changes: 2 additions & 2 deletions Providers/SegmentioProvider.m
Original file line number Diff line number Diff line change
Expand Up @@ -18,10 +18,10 @@ @interface SegmentioProvider ()
@implementation SegmentioProvider

#ifdef AR_SEGMENTIO_EXISTS
- (id)initWithIdentifier:(NSString *)identifier {
- (instancetype)initWithIdentifier:(NSString *)identifier {
return [self initWithIdentifier:identifier integrations:nil];
}
- (id)initWithIdentifier:(NSString *)identifier integrations:(NSArray *)integrations {
- (instancetype)initWithIdentifier:(NSString *)identifier integrations:(NSArray *)integrations {
if ((self = [super initWithIdentifier:identifier])) {
SEGAnalyticsConfiguration *config = [SEGAnalyticsConfiguration configurationWithWriteKey:identifier];
for (id integration in integrations) {
Expand Down
2 changes: 1 addition & 1 deletion Providers/SentryProvider.m
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@

@implementation SentryProvider

- (id)initWithIdentifier:(NSString *)identifier {
- (instancetype)initWithIdentifier:(NSString *)identifier {
NSAssert([RavenClient class], @"Raven is not included");
NSAssert([[RavenClient class] respondsToSelector:@selector(sharedClient)], @"Raven-ios library not installed correctly.");
[RavenClient setSharedClient:[RavenClient clientWithDSN:identifier]];
Expand Down
6 changes: 3 additions & 3 deletions Providers/SnowPlowProvider.h
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,8 @@
@class SnowplowEmitter;
@interface SnowplowProvider : ARAnalyticalProvider

- (id)initWithAddress:(NSString *)address;
- (id)initWithAddress:(NSString *)address appID:(NSString *)appID namespace:(NSString *)namespace;
- (id)initWithEmitter:(SnowplowEmitter *)emitter appID:(NSString *)appID namespace:(NSString *)namespace;
- (instancetype)initWithAddress:(NSString *)address;
- (instancetype)initWithAddress:(NSString *)address appID:(NSString *)appID namespace:(NSString *)namespace;
- (instancetype)initWithEmitter:(SnowplowEmitter *)emitter appID:(NSString *)appID namespace:(NSString *)namespace;

@end
6 changes: 3 additions & 3 deletions Providers/SnowPlowProvider.m
Original file line number Diff line number Diff line change
Expand Up @@ -15,19 +15,19 @@ @interface SnowplowProvider ()

@implementation SnowplowProvider

- (id)initWithAddress:(NSString *)address
- (instancetype)initWithAddress:(NSString *)address
{
NSString *bundleID = [[NSBundle mainBundle] bundleIdentifier];
return [self initWithAddress:address appID:bundleID namespace:nil];
}

- (id)initWithAddress:(NSString *)address appID:(NSString *)appID namespace:(NSString *)namespace
- (instancetype)initWithAddress:(NSString *)address appID:(NSString *)appID namespace:(NSString *)namespace
{
SnowplowEmitter *emitter = [[SnowplowEmitter alloc] initWithURLRequest:[NSURL URLWithString:address]];
return [self initWithEmitter:emitter appID:appID namespace:namespace];
}

- (id)initWithEmitter:(SnowplowEmitter *)emitter appID:(NSString *)appID namespace:(NSString *)namespace
- (instancetype)initWithEmitter:(SnowplowEmitter *)emitter appID:(NSString *)appID namespace:(NSString *)namespace
{
self = [super initWithIdentifier:appID];
if (!self) return nil;
Expand Down
Loading

0 comments on commit 8ec6a28

Please sign in to comment.