Facebook/react-native
1
0
Fork 0
mirror of https://github.com/facebook/react-native.git synced 2025-11-01 09:14:26 +00:00
Code Issues Packages Projects Releases Wiki Activity

rn | cxx | Move the definitions of TurboModule virtual methods to declaration, to allow for use in context with RTTI.

Browse Source 
Summary:
Not having this disallows including turbo module and extending in places where RTTI is enabled.
There is no additional includes or implementation changes - it merely allows for things to nicely link with other libraries.

Changelog: [General][Fixed] - Allow including TurboModule.h in mixed rtti/no-rtti environment, even if TurboModule.h/cpp is compiled without RTTI.

Reviewed By: appden

Differential Revision: D34637168

fbshipit-source-id: 2e5d9e546bdc5652f06436fec3b12f1aa9daab05
This commit is contained in:
Nikita Lutsenko
2022-03-04 19:04:36 -08:00
committed by Facebook GitHub Bot
parent 87ad1e8570
commit 1f87729697
3 changed files with 19 additions and 26 deletions
Download Patch File Download Diff File Expand all files Collapse all files
ReactAndroid/src/main/java/com/facebook/react/turbomodule/core/jni/Android.mk
+1 -1
View File
@@ -50,7 +50,7 @@ LOCAL_EXPORT_C_INCLUDES := $(LOCAL_PATH)
LOCAL_CFLAGS += -fexceptions -frtti -std=c++17 -Wall
LOCAL_SHARED_LIBRARIES = libfb libfbjni libreact_nativemodule_core
LOCAL_SHARED_LIBRARIES = libfb libfbjni libreact_nativemodule_core libjsi
LOCAL_STATIC_LIBRARIES = libcallinvokerholder libreactperfloggerjni