Skip to content

Commit cfb11ca

Browse files
Gal Shirinfacebook-github-bot
Gal Shirin
authored andcommitted
Fix xcodebeta build failure
Summary: Changelog: [iOS][Changed] - Removed methodName parameter that was used only for a warning message and moved the warning parameter to be calculated inline. Reviewed By: fkgozali Differential Revision: D34551444 fbshipit-source-id: 6ceba425b64df37b0dca7e222072f1836f151d83
1 parent 131844f commit cfb11ca

File tree

1 file changed

+3
-7
lines changed

1 file changed

+3
-7
lines changed

ReactCommon/react/nativemodule/core/platform/ios/RCTTurboModuleManager.mm

+3-7
Original file line numberDiff line numberDiff line change
@@ -702,18 +702,14 @@ - (BOOL)_requiresMainQueueSetup:(Class)moduleClass
702702

703703
BOOL requiresMainQueueSetup = hasConstantsToExport || hasCustomInit;
704704
if (requiresMainQueueSetup) {
705-
const char *methodName = "";
706-
if (hasConstantsToExport) {
707-
methodName = "constantsToExport";
708-
} else if (hasCustomInit) {
709-
methodName = "init";
710-
}
711705
RCTLogWarn(
712706
@"Module %@ requires main queue setup since it overrides `%s` but doesn't implement "
713707
"`requiresMainQueueSetup`. In a future release React Native will default to initializing all native modules "
714708
"on a background thread unless explicitly opted-out of.",
715709
moduleClass,
716-
methodName);
710+
hasConstantsToExport ? "constantsToExport"
711+
: hasCustomInit ? "init"
712+
: "");
717713
}
718714

719715
return requiresMainQueueSetup;

0 commit comments

Comments
 (0)