From 00e70354cadd2d33b985bc9c57c2e376eb4241a1 Mon Sep 17 00:00:00 2001 From: Ramanpreet Nara Date: Tue, 28 Nov 2023 13:17:22 -0800 Subject: [PATCH] CxxReactPackage: Use hungarian notation for hybridData (#41675) Summary: Pull Request resolved: https://github.com/facebook/react-native/pull/41675 FBJni expects the HybridData object to exist on the mHybridData property of the java object. So, we have to call this propery mHybridData. Otherwise, this fbjni class just won't work. Changelog: [Internal] Reviewed By: javache Differential Revision: D51550621 fbshipit-source-id: d169266474717f0a38799ede7c07af57461012b7 --- .../facebook/react/runtime/cxxreactpackage/CxxReactPackage.kt | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/runtime/cxxreactpackage/CxxReactPackage.kt b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/runtime/cxxreactpackage/CxxReactPackage.kt index 278d3d723c3..34a77fa2154 100644 --- a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/runtime/cxxreactpackage/CxxReactPackage.kt +++ b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/runtime/cxxreactpackage/CxxReactPackage.kt @@ -17,13 +17,13 @@ import com.facebook.soloader.SoLoader @UnstableReactNativeAPI() abstract class CxxReactPackage { - @DoNotStrip @SuppressWarnings("unused") private val hybridData: HybridData + @DoNotStrip @Suppress("NoHungarianNotation") private var mHybridData: HybridData? = initHybrid() protected abstract fun initHybrid(): HybridData protected constructor() { maybeLoadOtherSoLibraries() - hybridData = initHybrid() + mHybridData = initHybrid() } @Synchronized protected open fun maybeLoadOtherSoLibraries(): Unit {}