Mark initHybrid as @JvmStatic (#39755)

mdvacca · facebook-github-bot · commit 0371014a33c0 · 2023-10-02T13:10:32.000-07:00
Summary: Pull Request resolved: #39755 The goal of this diff is to fix: ``` JNI DETECTED ERROR IN APPLICATION: JNI NewGlobalRef called with pending exception java.lang.NoSuchMethodError: no static or non-static method "Lcom/facebook/react/jscexecutor/JSCExecutor;.initHybrid(Lcom/facebook/react/bridge/ReadableNativeMap;)Lcom/facebook/jni/HybridData;" ``` changelog: [internal] internal Reviewed By: luluwu2032 Differential Revision: D49831595 fbshipit-source-id: 9ce22cdccdd02af74edb27be2df72a469d3166c9
diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/jscexecutor/JSCExecutor.kt b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/jscexecutor/JSCExecutor.kt
@@ -31,6 +31,6 @@ class JSCExecutor internal constructor(jscConfig: ReadableNativeMap) :
       SoLoader.loadLibrary("jscexecutor")
     }
 
-    private external fun initHybrid(jscConfig: ReadableNativeMap): HybridData?
+    @JvmStatic private external fun initHybrid(jscConfig: ReadableNativeMap): HybridData?
   }
 }

Original file line number	Diff line number	Diff line change
`@@ -31,6 +31,6 @@ class JSCExecutor internal constructor(jscConfig: ReadableNativeMap) :`
`31`	`31`	`SoLoader.loadLibrary("jscexecutor")`
`32`	`32`	`}`
`33`	`33`
`34`		`- private external fun initHybrid(jscConfig: ReadableNativeMap): HybridData?`
	`34`	`+ @JvmStatic private external fun initHybrid(jscConfig: ReadableNativeMap): HybridData?`
`35`	`35`	`}`
`36`	`36`	`}`