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

fbshipit-source-id: 84ba72a43514cbf64270ac656c55c5a7aa082948
  • Loading branch information
Lulu Wu authored and cortinico committed Oct 27, 2023
1 parent abfdf2c commit c2d4aad
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 c2d4aad

Please sign in to comment.