Skip to content

Commit 2b2c69a

Browse files
RSNarafacebook-github-bot
authored andcommitted
Prevent TurboModuleManagerDelegate.mHybridData from being dead-code eliminated
Summary: mHybridData should have a SuppressWarnings("unsed") annotation on it. Furthermore, we should add a DoNotStrip annotation, to make sure that the HybridData object doesn't get removed. We do the same in TurboModuleManager: https://www.internalfb.com/intern/diffusion/FBS/browsefile/master/xplat/js/react-native-github/ReactAndroid/src/main/java/com/facebook/react/turbomodule/core/TurboModuleManager.java?commit=5e1cd0c2ed84ff8e718473ff26145733868ebaba&lines=46-48 Changelog: [Internal] Created from Diffusion's 'Open in Editor' feature. Reviewed By: fkgozali Differential Revision: D26958686 fbshipit-source-id: 83c0626c0681966dfcda7f6e4fb4cbeac8340b3f
1 parent 23d9bf1 commit 2b2c69a

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

ReactAndroid/src/main/java/com/facebook/react/turbomodule/core/TurboModuleManagerDelegate.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,13 +9,16 @@
99

1010
import androidx.annotation.Nullable;
1111
import com.facebook.jni.HybridData;
12+
import com.facebook.proguard.annotations.DoNotStrip;
1213
import com.facebook.react.bridge.CxxModuleWrapper;
1314
import com.facebook.react.turbomodule.core.interfaces.TurboModule;
1415
import com.facebook.soloader.SoLoader;
1516
import java.util.ArrayList;
1617
import java.util.List;
1718

1819
public abstract class TurboModuleManagerDelegate {
20+
@DoNotStrip
21+
@SuppressWarnings("unused")
1922
private final HybridData mHybridData;
2023

2124
private static volatile boolean sIsSoLibraryLoaded;

0 commit comments

Comments
 (0)