Skip to content

Commit

Permalink
Add @DoNotStripAny to JSTimerExecutor
Browse files Browse the repository at this point in the history
Summary:
RNTester release build is crashing as following, this diff add DoNotStripAny to JSTimerExecutor to avoid over-stripping for release build.


 {F1124087084}

Changelog:
[Android][Changed] -  Add DoNotStripAny to JSTimerExecutor

Reviewed By: cortinico

Differential Revision: D50410412
  • Loading branch information
luluwu2032 authored and facebook-github-bot committed Oct 18, 2023
1 parent 87dbe44 commit d051b27
Showing 1 changed file with 3 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -9,20 +9,21 @@

import com.facebook.infer.annotation.Nullsafe;
import com.facebook.jni.HybridData;
import com.facebook.jni.annotations.DoNotStrip;
import com.facebook.jni.annotations.DoNotStripAny;
import com.facebook.react.bridge.WritableArray;
import com.facebook.react.bridge.WritableNativeArray;
import com.facebook.react.modules.core.JavaScriptTimerExecutor;
import com.facebook.soloader.SoLoader;

@Nullsafe(Nullsafe.Mode.LOCAL)
@DoNotStripAny
class JSTimerExecutor implements JavaScriptTimerExecutor {

static {
SoLoader.loadLibrary("rninstance");
}

@DoNotStrip private final HybridData mHybridData;
private final HybridData mHybridData;

public JSTimerExecutor(HybridData hybridData) {
mHybridData = hybridData;
Expand Down

0 comments on commit d051b27

Please sign in to comment.