Skip to content

Commit

Permalink
Fix crash for Hermes Release due to HermesExecutor migration
Browse files Browse the repository at this point in the history
Summary:
RN-Tester is currently instacrashing on release due to a migration to Kotlin for HermesExecutor
This fixes it.

Changelog:
[Internal] [Changed] - Fix crash for Hermes Release due to HermesExecutor migration

Differential Revision: D68151666
  • Loading branch information
cortinico authored and facebook-github-bot committed Jan 14, 2025
1 parent d95909e commit 766d6b1
Showing 1 changed file with 7 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -8,10 +8,13 @@
package com.facebook.hermes.reactexecutor

import com.facebook.jni.HybridData
import com.facebook.jni.annotations.DoNotStripAny
import com.facebook.proguard.annotations.DoNotStrip
import com.facebook.react.bridge.JavaScriptExecutor
import com.facebook.react.common.build.ReactBuildConfig
import com.facebook.soloader.SoLoader

@DoNotStripAny
public class HermesExecutor
internal constructor(config: RuntimeConfig?, enableDebugger: Boolean, debuggerName: String) :
JavaScriptExecutor(
Expand Down Expand Up @@ -39,11 +42,15 @@ internal constructor(config: RuntimeConfig?, enableDebugger: Boolean, debuggerNa
}
}

@DoNotStrip
@JvmStatic
private external fun initHybridDefaultConfig(
enableDebugger: Boolean,
debuggerName: String
): HybridData?

@DoNotStrip
@JvmStatic
private external fun initHybrid(
enableDebugger: Boolean,
debuggerName: String,
Expand Down

0 comments on commit 766d6b1

Please sign in to comment.