Skip to content

Commit 8b2407a

Browse files
rntester Android > Don't call rncore_ModuleProvider(name, params) twice (#40758)
Summary: This is a follow up to #39987 As we already call ``` return rncore_ModuleProvider(name, params); ``` in: https://github.com/facebook/react-native/blob/main/packages/react-native/ReactAndroid/src/main/jni/react/newarchdefaults/DefaultTurboModuleManagerDelegate.cpp#L55 we don't have to do it again in: https://github.com/facebook/react-native/blob/main/packages/rn-tester/android/app/src/main/jni/OnLoad.cpp#L53 Changelog: [Internal] Reviewed By: cortinico Differential Revision: D50109991
1 parent b13ea17 commit 8b2407a

File tree

1 file changed

+1
-2
lines changed
  • packages/rn-tester/android/app/src/main/jni

1 file changed

+1
-2
lines changed

packages/rn-tester/android/app/src/main/jni/OnLoad.cpp

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,6 @@
1414
#include <react/renderer/componentregistry/ComponentDescriptorProviderRegistry.h>
1515
#include <react/renderer/components/AppSpecs/ComponentDescriptors.h>
1616
#include <react/renderer/components/legacyviewmanagerinterop/UnstableLegacyViewManagerInteropComponentDescriptor.h>
17-
#include <rncore.h>
1817

1918
namespace facebook {
2019
namespace react {
@@ -50,7 +49,7 @@ std::shared_ptr<TurboModule> javaModuleProvider(
5049
if (module != nullptr) {
5150
return module;
5251
};
53-
return rncore_ModuleProvider(name, params);
52+
return nullptr;
5453
}
5554

5655
} // namespace react

0 commit comments

Comments
 (0)