From d50c9068a927ec1fa0c574a4483dc7c03dca75f5 Mon Sep 17 00:00:00 2001 From: Kevin Gozali Date: Fri, 5 Jan 2024 09:11:41 -0800 Subject: [PATCH] iOS: Fixed up remaining RCT_USE_HERMES usage --> USE_HERMES (#42148) Summary: Pull Request resolved: https://github.com/facebook/react-native/pull/42148 https://github.com/facebook/react-native/pull/41625 cleaned up some usages, but there are some remaining ones. Changelog: [iOS][Fixed] Further cleaned up RCT_USE_HERMES Reviewed By: cipolleschi Differential Revision: D52555309 fbshipit-source-id: e19d10de339636eca2f4762a78cebb282d0427d9 --- .../Libraries/AppDelegate/RCTAppSetupUtils.h | 11 ++++------- .../Libraries/AppDelegate/RCTAppSetupUtils.mm | 8 ++++---- 2 files changed, 8 insertions(+), 11 deletions(-) diff --git a/packages/react-native/Libraries/AppDelegate/RCTAppSetupUtils.h b/packages/react-native/Libraries/AppDelegate/RCTAppSetupUtils.h index af14ba93b91..095a002ee74 100644 --- a/packages/react-native/Libraries/AppDelegate/RCTAppSetupUtils.h +++ b/packages/react-native/Libraries/AppDelegate/RCTAppSetupUtils.h @@ -15,18 +15,15 @@ #import -#ifndef RCT_USE_HERMES +#if USE_HERMES #if __has_include() #import -#define RCT_USE_HERMES 1 #elif __has_include() #import -#define RCT_USE_HERMES 1 -#else +#endif +#else // USE_HERMES #import -#define RCT_USE_HERMES 0 -#endif -#endif +#endif // USE_HERMES #import diff --git a/packages/react-native/Libraries/AppDelegate/RCTAppSetupUtils.mm b/packages/react-native/Libraries/AppDelegate/RCTAppSetupUtils.mm index 68b1f417995..865cac04ff4 100644 --- a/packages/react-native/Libraries/AppDelegate/RCTAppSetupUtils.mm +++ b/packages/react-native/Libraries/AppDelegate/RCTAppSetupUtils.mm @@ -107,11 +107,11 @@ std::unique_ptr RCTAppSetupDefaultJsExecutor [turboModuleManager moduleForName:"RCTDevMenu"]; #endif // end RCT_DEV -#if RCT_USE_HERMES +#if USE_HERMES return std::make_unique( #else return std::make_unique( -#endif // end RCT_USE_HERMES +#endif // USE_HERMES facebook::react::RCTJSIExecutorRuntimeInstaller( [turboModuleManager, bridge, runtimeScheduler](facebook::jsi::Runtime &runtime) { if (!bridge || !turboModuleManager) { @@ -128,11 +128,11 @@ std::unique_ptr RCTAppSetupJsExecutorFactory RCTBridge *bridge, const std::shared_ptr &runtimeScheduler) { -#if RCT_USE_HERMES +#if USE_HERMES return std::make_unique( #else return std::make_unique( -#endif // end RCT_USE_HERMES +#endif // USE_HERMES facebook::react::RCTJSIExecutorRuntimeInstaller([bridge, runtimeScheduler](facebook::jsi::Runtime &runtime) { if (!bridge) { return;