Skip to content

Commit

Permalink
Actually fix java.lang.NoSuchMethodError for Bridgeless (#41081)
Browse files Browse the repository at this point in the history
Summary:
Pull Request resolved: #41081

It turns out that Bridgeless for RN-Tester release is still broken.
This fixes it by making sure we actually `DoNotStrip` the missing constructor

Changelog:
[Android] [Fixed] - Fix crash with `java.lang.NoSuchMethodError` for Bridgeless

Reviewed By: RSNara

Differential Revision: D50455967

fbshipit-source-id: eae971fceeb863d8a400e9de1d2467637d59d2b0
  • Loading branch information
cortinico authored and facebook-github-bot committed Oct 23, 2023
1 parent 0806ad7 commit b22f566
Showing 1 changed file with 3 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@
import com.facebook.infer.annotation.Nullsafe;
import com.facebook.jni.HybridData;
import com.facebook.jni.annotations.DoNotStripAny;
import com.facebook.proguard.annotations.DoNotStrip;
import com.facebook.react.bridge.WritableArray;
import com.facebook.react.bridge.WritableNativeArray;
import com.facebook.react.modules.core.JavaScriptTimerExecutor;
Expand All @@ -23,8 +24,9 @@ class JSTimerExecutor implements JavaScriptTimerExecutor {
SoLoader.loadLibrary("rninstance");
}

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

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

0 comments on commit b22f566

Please sign in to comment.