Skip to content

Commit 770776d

Browse files
authored
update comments (#11968)
1 parent a15117c commit 770776d

File tree

2 files changed

+6
-2
lines changed

2 files changed

+6
-2
lines changed

Crashlytics/Crashlytics/Components/FIRCLSApplication.h

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -44,6 +44,8 @@ NSString* FIRCLSApplicationGetSDKBundleID(void);
4444
/**
4545
* Returns the platform identifier, either: ios, mac, or tvos.
4646
* Catalyst apps are treated as mac.
47+
* This is a legacy function, for platform identificaiton please use
48+
* FIRCLSApplicationGetFirebasePlatform.
4749
*/
4850
NSString* FIRCLSApplicationGetPlatform(void);
4951

Crashlytics/Crashlytics/Components/FIRCLSApplication.m

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,8 @@
3737
[@"com.google.firebase.crashlytics." stringByAppendingString:FIRCLSApplicationGetPlatform()];
3838
}
3939

40+
// Legacy function, we use FIRCLSApplicationGetFirebasePlatform now for platform specification.
41+
// Can't clean the code since some endpoints setup depend on this function.
4042
NSString* FIRCLSApplicationGetPlatform(void) {
4143
#if defined(TARGET_OS_MACCATALYST) && TARGET_OS_MACCATALYST
4244
return @"mac";
@@ -47,9 +49,9 @@
4749
#elif TARGET_OS_TV
4850
return @"tvos";
4951
#elif TARGET_OS_WATCH
50-
return @"ios"; // TODO: temporarily use iOS until Firebase can add watchos to the backend
52+
return @"ios";
5153
#elif defined(TARGET_OS_VISION) && TARGET_OS_VISION
52-
return @"ios"; // TODO: temporarily use iOS until Firebase can add visionos to the backend
54+
return @"ios";
5355
#endif
5456
}
5557

0 commit comments

Comments
 (0)