From cfb11ca2f67c59c090b8a58b2b7bdaacef0e19df Mon Sep 17 00:00:00 2001
From: Gal Shirin <gls@fb.com>
Date: Tue, 1 Mar 2022 23:56:55 -0800
Subject: [PATCH] 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
---
 .../core/platform/ios/RCTTurboModuleManager.mm         | 10 +++-------
 1 file changed, 3 insertions(+), 7 deletions(-)

diff --git a/ReactCommon/react/nativemodule/core/platform/ios/RCTTurboModuleManager.mm b/ReactCommon/react/nativemodule/core/platform/ios/RCTTurboModuleManager.mm
index 67227a0a93..d795920309 100644
--- a/ReactCommon/react/nativemodule/core/platform/ios/RCTTurboModuleManager.mm
+++ b/ReactCommon/react/nativemodule/core/platform/ios/RCTTurboModuleManager.mm
@@ -702,18 +702,14 @@ static Class getFallbackClassFromName(const char *name)
 
   BOOL requiresMainQueueSetup = hasConstantsToExport || hasCustomInit;
   if (requiresMainQueueSetup) {
-    const char *methodName = "";
-    if (hasConstantsToExport) {
-      methodName = "constantsToExport";
-    } else if (hasCustomInit) {
-      methodName = "init";
-    }
     RCTLogWarn(
         @"Module %@ requires main queue setup since it overrides `%s` but doesn't implement "
          "`requiresMainQueueSetup`. In a future release React Native will default to initializing all native modules "
          "on a background thread unless explicitly opted-out of.",
         moduleClass,
-        methodName);
+        hasConstantsToExport ? "constantsToExport"
+            : hasCustomInit  ? "init"
+                             : "");
   }
 
   return requiresMainQueueSetup;