Remove dead RAMBundleRegistry::singleBundleRegistry factory

sammy-SC · meta-codesync[bot] · commit b4de836f57b9 · 2026-06-10T09:17:52.000-07:00
Summary: X-link: #57143 `RAMBundleRegistry::singleBundleRegistry` was a static factory that wrapped the public `RAMBundleRegistry` constructor, but it had no callers anywhere. Objects are constructed via the public constructor directly. This removes the orphaned factory; the sibling `multipleBundlesRegistry`, the constructor, `MAIN_BUNDLE_ID`, `registerBundle`, `getModule`, and `getBundle` are all left intact. Changelog: [Internal] Reviewed By: mdvacca Differential Revision: D108012903 fbshipit-source-id: 5d8f18ebce2bb19abd124dbb7c9eda60baa0f272
diff --git a/packages/react-native/ReactCommon/cxxreact/RAMBundleRegistry.cpp b/packages/react-native/ReactCommon/cxxreact/RAMBundleRegistry.cpp
@@ -20,11 +20,6 @@ namespace facebook::react {
 constexpr uint32_t RAMBundleRegistry::MAIN_BUNDLE_ID;
 #pragma clang diagnostic pop
 
-std::unique_ptr<RAMBundleRegistry> RAMBundleRegistry::singleBundleRegistry(
-    std::unique_ptr<JSModulesUnbundle> mainBundle) {
-  return std::make_unique<RAMBundleRegistry>(std::move(mainBundle));
-}
-
 std::unique_ptr<RAMBundleRegistry> RAMBundleRegistry::multipleBundlesRegistry(
     std::unique_ptr<JSModulesUnbundle> mainBundle,
     std::function<std::unique_ptr<JSModulesUnbundle>(std::string)> factory) {
diff --git a/packages/react-native/ReactCommon/cxxreact/RAMBundleRegistry.h b/packages/react-native/ReactCommon/cxxreact/RAMBundleRegistry.h
@@ -27,7 +27,6 @@ class RN_EXPORT [[deprecated("This API will be removed along with the legacy arc
  public:
   constexpr static uint32_t MAIN_BUNDLE_ID = 0;
 
-  static std::unique_ptr<RAMBundleRegistry> singleBundleRegistry(std::unique_ptr<JSModulesUnbundle> mainBundle);
   static std::unique_ptr<RAMBundleRegistry> multipleBundlesRegistry(
       std::unique_ptr<JSModulesUnbundle> mainBundle,
       std::function<std::unique_ptr<JSModulesUnbundle>(std::string)> factory);
diff --git a/scripts/cxx-api/api-snapshots/ReactAndroidDebugCxx.api b/scripts/cxx-api/api-snapshots/ReactAndroidDebugCxx.api
@@ -4180,7 +4180,6 @@ class facebook::react::RAMBundleRegistry {
   public facebook::react::RAMBundleRegistry& operator=(facebook::react::RAMBundleRegistry&&) = default;
   public static constexpr uint32_t MAIN_BUNDLE_ID;
   public static std::unique_ptr<facebook::react::RAMBundleRegistry> multipleBundlesRegistry(std::unique_ptr<facebook::react::JSModulesUnbundle> mainBundle, std::function<std::unique_ptr<facebook::react::JSModulesUnbundle>(std::string)> factory);
-  public static std::unique_ptr<facebook::react::RAMBundleRegistry> singleBundleRegistry(std::unique_ptr<facebook::react::JSModulesUnbundle> mainBundle);
   public virtual ~RAMBundleRegistry() = default;
   public void registerBundle(uint32_t bundleId, std::string bundlePath);
 }
diff --git a/scripts/cxx-api/api-snapshots/ReactAndroidReleaseCxx.api b/scripts/cxx-api/api-snapshots/ReactAndroidReleaseCxx.api
@@ -4177,7 +4177,6 @@ class facebook::react::RAMBundleRegistry {
   public facebook::react::RAMBundleRegistry& operator=(facebook::react::RAMBundleRegistry&&) = default;
   public static constexpr uint32_t MAIN_BUNDLE_ID;
   public static std::unique_ptr<facebook::react::RAMBundleRegistry> multipleBundlesRegistry(std::unique_ptr<facebook::react::JSModulesUnbundle> mainBundle, std::function<std::unique_ptr<facebook::react::JSModulesUnbundle>(std::string)> factory);
-  public static std::unique_ptr<facebook::react::RAMBundleRegistry> singleBundleRegistry(std::unique_ptr<facebook::react::JSModulesUnbundle> mainBundle);
   public virtual ~RAMBundleRegistry() = default;
   public void registerBundle(uint32_t bundleId, std::string bundlePath);
 }
diff --git a/scripts/cxx-api/api-snapshots/ReactAppleDebugCxx.api b/scripts/cxx-api/api-snapshots/ReactAppleDebugCxx.api
@@ -6361,7 +6361,6 @@ class facebook::react::RAMBundleRegistry {
   public facebook::react::RAMBundleRegistry& operator=(facebook::react::RAMBundleRegistry&&) = default;
   public static constexpr uint32_t MAIN_BUNDLE_ID;
   public static std::unique_ptr<facebook::react::RAMBundleRegistry> multipleBundlesRegistry(std::unique_ptr<facebook::react::JSModulesUnbundle> mainBundle, std::function<std::unique_ptr<facebook::react::JSModulesUnbundle>(std::string)> factory);
-  public static std::unique_ptr<facebook::react::RAMBundleRegistry> singleBundleRegistry(std::unique_ptr<facebook::react::JSModulesUnbundle> mainBundle);
   public virtual ~RAMBundleRegistry() = default;
   public void registerBundle(uint32_t bundleId, std::string bundlePath);
 }
diff --git a/scripts/cxx-api/api-snapshots/ReactAppleReleaseCxx.api b/scripts/cxx-api/api-snapshots/ReactAppleReleaseCxx.api
@@ -6358,7 +6358,6 @@ class facebook::react::RAMBundleRegistry {
   public facebook::react::RAMBundleRegistry& operator=(facebook::react::RAMBundleRegistry&&) = default;
   public static constexpr uint32_t MAIN_BUNDLE_ID;
   public static std::unique_ptr<facebook::react::RAMBundleRegistry> multipleBundlesRegistry(std::unique_ptr<facebook::react::JSModulesUnbundle> mainBundle, std::function<std::unique_ptr<facebook::react::JSModulesUnbundle>(std::string)> factory);
-  public static std::unique_ptr<facebook::react::RAMBundleRegistry> singleBundleRegistry(std::unique_ptr<facebook::react::JSModulesUnbundle> mainBundle);
   public virtual ~RAMBundleRegistry() = default;
   public void registerBundle(uint32_t bundleId, std::string bundlePath);
 }
diff --git a/scripts/cxx-api/api-snapshots/ReactCommonDebugCxx.api b/scripts/cxx-api/api-snapshots/ReactCommonDebugCxx.api
@@ -2761,7 +2761,6 @@ class facebook::react::RAMBundleRegistry {
   public facebook::react::RAMBundleRegistry& operator=(facebook::react::RAMBundleRegistry&&) = default;
   public static constexpr uint32_t MAIN_BUNDLE_ID;
   public static std::unique_ptr<facebook::react::RAMBundleRegistry> multipleBundlesRegistry(std::unique_ptr<facebook::react::JSModulesUnbundle> mainBundle, std::function<std::unique_ptr<facebook::react::JSModulesUnbundle>(std::string)> factory);
-  public static std::unique_ptr<facebook::react::RAMBundleRegistry> singleBundleRegistry(std::unique_ptr<facebook::react::JSModulesUnbundle> mainBundle);
   public virtual ~RAMBundleRegistry() = default;
   public void registerBundle(uint32_t bundleId, std::string bundlePath);
 }
diff --git a/scripts/cxx-api/api-snapshots/ReactCommonReleaseCxx.api b/scripts/cxx-api/api-snapshots/ReactCommonReleaseCxx.api
@@ -2758,7 +2758,6 @@ class facebook::react::RAMBundleRegistry {
   public facebook::react::RAMBundleRegistry& operator=(facebook::react::RAMBundleRegistry&&) = default;
   public static constexpr uint32_t MAIN_BUNDLE_ID;
   public static std::unique_ptr<facebook::react::RAMBundleRegistry> multipleBundlesRegistry(std::unique_ptr<facebook::react::JSModulesUnbundle> mainBundle, std::function<std::unique_ptr<facebook::react::JSModulesUnbundle>(std::string)> factory);
-  public static std::unique_ptr<facebook::react::RAMBundleRegistry> singleBundleRegistry(std::unique_ptr<facebook::react::JSModulesUnbundle> mainBundle);
   public virtual ~RAMBundleRegistry() = default;
   public void registerBundle(uint32_t bundleId, std::string bundlePath);
 }

Original file line number	Diff line number	Diff line change
`@@ -4180,7 +4180,6 @@ class facebook::react::RAMBundleRegistry {`
`4180`	`4180`	`public facebook::react::RAMBundleRegistry& operator=(facebook::react::RAMBundleRegistry&&) = default;`
`4181`	`4181`	`public static constexpr uint32_t MAIN_BUNDLE_ID;`
`4182`	`4182`	`public static std::unique_ptr<facebook::react::RAMBundleRegistry> multipleBundlesRegistry(std::unique_ptr<facebook::react::JSModulesUnbundle> mainBundle, std::function<std::unique_ptr<facebook::react::JSModulesUnbundle>(std::string)> factory);`
`4183`		`- public static std::unique_ptr<facebook::react::RAMBundleRegistry> singleBundleRegistry(std::unique_ptr<facebook::react::JSModulesUnbundle> mainBundle);`
`4184`	`4183`	`public virtual ~RAMBundleRegistry() = default;`
`4185`	`4184`	`public void registerBundle(uint32_t bundleId, std::string bundlePath);`
`4186`	`4185`	`}`
Original file line number	Diff line number	Diff line change
`@@ -4177,7 +4177,6 @@ class facebook::react::RAMBundleRegistry {`
`4177`	`4177`	`public facebook::react::RAMBundleRegistry& operator=(facebook::react::RAMBundleRegistry&&) = default;`
`4178`	`4178`	`public static constexpr uint32_t MAIN_BUNDLE_ID;`
`4179`	`4179`	`public static std::unique_ptr<facebook::react::RAMBundleRegistry> multipleBundlesRegistry(std::unique_ptr<facebook::react::JSModulesUnbundle> mainBundle, std::function<std::unique_ptr<facebook::react::JSModulesUnbundle>(std::string)> factory);`
`4180`		`- public static std::unique_ptr<facebook::react::RAMBundleRegistry> singleBundleRegistry(std::unique_ptr<facebook::react::JSModulesUnbundle> mainBundle);`
`4181`	`4180`	`public virtual ~RAMBundleRegistry() = default;`
`4182`	`4181`	`public void registerBundle(uint32_t bundleId, std::string bundlePath);`
`4183`	`4182`	`}`
Original file line number	Diff line number	Diff line change
`@@ -6361,7 +6361,6 @@ class facebook::react::RAMBundleRegistry {`
`6361`	`6361`	`public facebook::react::RAMBundleRegistry& operator=(facebook::react::RAMBundleRegistry&&) = default;`
`6362`	`6362`	`public static constexpr uint32_t MAIN_BUNDLE_ID;`
`6363`	`6363`	`public static std::unique_ptr<facebook::react::RAMBundleRegistry> multipleBundlesRegistry(std::unique_ptr<facebook::react::JSModulesUnbundle> mainBundle, std::function<std::unique_ptr<facebook::react::JSModulesUnbundle>(std::string)> factory);`
`6364`		`- public static std::unique_ptr<facebook::react::RAMBundleRegistry> singleBundleRegistry(std::unique_ptr<facebook::react::JSModulesUnbundle> mainBundle);`
`6365`	`6364`	`public virtual ~RAMBundleRegistry() = default;`
`6366`	`6365`	`public void registerBundle(uint32_t bundleId, std::string bundlePath);`
`6367`	`6366`	`}`
Original file line number	Diff line number	Diff line change
`@@ -6358,7 +6358,6 @@ class facebook::react::RAMBundleRegistry {`
`6358`	`6358`	`public facebook::react::RAMBundleRegistry& operator=(facebook::react::RAMBundleRegistry&&) = default;`
`6359`	`6359`	`public static constexpr uint32_t MAIN_BUNDLE_ID;`
`6360`	`6360`	`public static std::unique_ptr<facebook::react::RAMBundleRegistry> multipleBundlesRegistry(std::unique_ptr<facebook::react::JSModulesUnbundle> mainBundle, std::function<std::unique_ptr<facebook::react::JSModulesUnbundle>(std::string)> factory);`
`6361`		`- public static std::unique_ptr<facebook::react::RAMBundleRegistry> singleBundleRegistry(std::unique_ptr<facebook::react::JSModulesUnbundle> mainBundle);`
`6362`	`6361`	`public virtual ~RAMBundleRegistry() = default;`
`6363`	`6362`	`public void registerBundle(uint32_t bundleId, std::string bundlePath);`
`6364`	`6363`	`}`
Original file line number	Diff line number	Diff line change
`@@ -2761,7 +2761,6 @@ class facebook::react::RAMBundleRegistry {`
`2761`	`2761`	`public facebook::react::RAMBundleRegistry& operator=(facebook::react::RAMBundleRegistry&&) = default;`
`2762`	`2762`	`public static constexpr uint32_t MAIN_BUNDLE_ID;`
`2763`	`2763`	`public static std::unique_ptr<facebook::react::RAMBundleRegistry> multipleBundlesRegistry(std::unique_ptr<facebook::react::JSModulesUnbundle> mainBundle, std::function<std::unique_ptr<facebook::react::JSModulesUnbundle>(std::string)> factory);`
`2764`		`- public static std::unique_ptr<facebook::react::RAMBundleRegistry> singleBundleRegistry(std::unique_ptr<facebook::react::JSModulesUnbundle> mainBundle);`
`2765`	`2764`	`public virtual ~RAMBundleRegistry() = default;`
`2766`	`2765`	`public void registerBundle(uint32_t bundleId, std::string bundlePath);`
`2767`	`2766`	`}`
Original file line number	Diff line number	Diff line change
`@@ -2758,7 +2758,6 @@ class facebook::react::RAMBundleRegistry {`
`2758`	`2758`	`public facebook::react::RAMBundleRegistry& operator=(facebook::react::RAMBundleRegistry&&) = default;`
`2759`	`2759`	`public static constexpr uint32_t MAIN_BUNDLE_ID;`
`2760`	`2760`	`public static std::unique_ptr<facebook::react::RAMBundleRegistry> multipleBundlesRegistry(std::unique_ptr<facebook::react::JSModulesUnbundle> mainBundle, std::function<std::unique_ptr<facebook::react::JSModulesUnbundle>(std::string)> factory);`
`2761`		`- public static std::unique_ptr<facebook::react::RAMBundleRegistry> singleBundleRegistry(std::unique_ptr<facebook::react::JSModulesUnbundle> mainBundle);`
`2762`	`2761`	`public virtual ~RAMBundleRegistry() = default;`
`2763`	`2762`	`public void registerBundle(uint32_t bundleId, std::string bundlePath);`
`2764`	`2763`	`}`