From cbdbb474675bb6fbd5857873234d825c52ca16b3 Mon Sep 17 00:00:00 2001
From: Ramanpreet Nara <ramanpreet@meta.com>
Date: Tue, 14 Mar 2023 21:16:22 -0700
Subject: [PATCH] Refactor: Move ModuleProvider param to the end in
 TurboModuleBinding::install()

Summary:
For the TurboModule interop layer, we'll pass in a second ModuleProvider. We'll default that second parameter to null, when the interop layer is disabled. This change will make that easier.

Changelog: [General][Changed] - Re-organize the parameters of TurboModuleBinding::install()

Reviewed By: javache, cortinico

Differential Revision: D43993199

fbshipit-source-id: 2d339cab1f9bfe481f4b72401caa203cd7536da1
---
 .../turbomodule/ReactCommon/TurboModuleManager.cpp   |  2 +-
 .../core/ReactCommon/TurboModuleBinding.cpp          | 12 ++++++------
 .../core/ReactCommon/TurboModuleBinding.h            | 10 +++++-----
 .../ios/ReactCommon/RCTTurboModuleManager.mm         |  2 +-
 4 files changed, 13 insertions(+), 13 deletions(-)

diff --git a/ReactAndroid/src/main/jni/react/turbomodule/ReactCommon/TurboModuleManager.cpp b/ReactAndroid/src/main/jni/react/turbomodule/ReactCommon/TurboModuleManager.cpp
index 437d3384d18..7fa0f863b9e 100644
--- a/ReactAndroid/src/main/jni/react/turbomodule/ReactCommon/TurboModuleManager.cpp
+++ b/ReactAndroid/src/main/jni/react/turbomodule/ReactCommon/TurboModuleManager.cpp
@@ -152,7 +152,7 @@ void TurboModuleManager::installJSIBindings() {
     TurboModuleBindingMode bindingMode = static_cast<TurboModuleBindingMode>(
         getFeatureFlagValue("turboModuleBindingMode"));
     TurboModuleBinding::install(
-        runtime, std::move(turboModuleProvider), bindingMode);
+        runtime, bindingMode, std::move(turboModuleProvider));
   });
 }
 
diff --git a/ReactCommon/react/nativemodule/core/ReactCommon/TurboModuleBinding.cpp b/ReactCommon/react/nativemodule/core/ReactCommon/TurboModuleBinding.cpp
index 94da446666f..cc08d03c028 100644
--- a/ReactCommon/react/nativemodule/core/ReactCommon/TurboModuleBinding.cpp
+++ b/ReactCommon/react/nativemodule/core/ReactCommon/TurboModuleBinding.cpp
@@ -23,14 +23,14 @@ namespace react {
  */
 
 TurboModuleBinding::TurboModuleBinding(
-    const TurboModuleProviderFunctionType &&moduleProvider,
-    TurboModuleBindingMode bindingMode)
-    : moduleProvider_(std::move(moduleProvider)), bindingMode_(bindingMode) {}
+    TurboModuleBindingMode bindingMode,
+    TurboModuleProviderFunctionType &&moduleProvider)
+    : bindingMode_(bindingMode), moduleProvider_(std::move(moduleProvider)) {}
 
 void TurboModuleBinding::install(
     jsi::Runtime &runtime,
-    const TurboModuleProviderFunctionType &&moduleProvider,
-    TurboModuleBindingMode bindingMode) {
+    TurboModuleBindingMode bindingMode,
+    TurboModuleProviderFunctionType &&moduleProvider) {
   runtime.global().setProperty(
       runtime,
       "__turboModuleProxy",
@@ -39,7 +39,7 @@ void TurboModuleBinding::install(
           jsi::PropNameID::forAscii(runtime, "__turboModuleProxy"),
           1,
           [binding =
-               TurboModuleBinding(std::move(moduleProvider), bindingMode)](
+               TurboModuleBinding(bindingMode, std::move(moduleProvider))](
               jsi::Runtime &rt,
               const jsi::Value &thisVal,
               const jsi::Value *args,
diff --git a/ReactCommon/react/nativemodule/core/ReactCommon/TurboModuleBinding.h b/ReactCommon/react/nativemodule/core/ReactCommon/TurboModuleBinding.h
index dcf8eaaa2d8..cbcd729b961 100644
--- a/ReactCommon/react/nativemodule/core/ReactCommon/TurboModuleBinding.h
+++ b/ReactCommon/react/nativemodule/core/ReactCommon/TurboModuleBinding.h
@@ -32,13 +32,13 @@ class TurboModuleBinding {
    */
   static void install(
       jsi::Runtime &runtime,
-      const TurboModuleProviderFunctionType &&moduleProvider,
-      TurboModuleBindingMode bindingMode);
+      TurboModuleBindingMode bindingMode,
+      TurboModuleProviderFunctionType &&moduleProvider);
 
  private:
   TurboModuleBinding(
-      const TurboModuleProviderFunctionType &&moduleProvider,
-      TurboModuleBindingMode bindingMode);
+      TurboModuleBindingMode bindingMode,
+      TurboModuleProviderFunctionType &&moduleProvider);
   virtual ~TurboModuleBinding();
 
   /**
@@ -48,8 +48,8 @@ class TurboModuleBinding {
   jsi::Value getModule(jsi::Runtime &runtime, const std::string &moduleName)
       const;
 
-  TurboModuleProviderFunctionType moduleProvider_;
   TurboModuleBindingMode bindingMode_;
+  TurboModuleProviderFunctionType moduleProvider_;
 };
 
 } // namespace react
diff --git a/ReactCommon/react/nativemodule/core/platform/ios/ReactCommon/RCTTurboModuleManager.mm b/ReactCommon/react/nativemodule/core/platform/ios/ReactCommon/RCTTurboModuleManager.mm
index 522247f1f7c..2a937891011 100644
--- a/ReactCommon/react/nativemodule/core/platform/ios/ReactCommon/RCTTurboModuleManager.mm
+++ b/ReactCommon/react/nativemodule/core/platform/ios/ReactCommon/RCTTurboModuleManager.mm
@@ -741,7 +741,7 @@ static Class getFallbackClassFromName(const char *name)
   };
 
   runtimeExecutor([turboModuleProvider = std::move(turboModuleProvider)](jsi::Runtime &runtime) {
-    TurboModuleBinding::install(runtime, std::move(turboModuleProvider), sTurboModuleBindingMode);
+    TurboModuleBinding::install(runtime, sTurboModuleBindingMode, std::move(turboModuleProvider));
   });
 }