Skip to content

Commit 7730899

Browse files
mdvaccameta-codesync[bot]
authored andcommitted
Inline true for fabricEnabled in DefaultNewArchitectureEntryPoint (#56882)
Summary: Pull Request resolved: #56882 Now that `fabricEnabled` always returns `true`, delete the `privateFabricEnabled` backing field. Remove the redundant assignment in `load()` and inline `true` at its remaining usage in `isConfigurationValid` from `loadWithFeatureFlags`. Behavior is unchanged. Changelog: [Internal] Reviewed By: alanleedev, fkgozali Differential Revision: D105639307 fbshipit-source-id: 0c229ece1b50965cf9b6dcf8ec744d5373f8a91d
1 parent 58cd1bf commit 7730899

1 file changed

Lines changed: 1 addition & 4 deletions

File tree

packages/react-native/ReactAndroid/src/main/java/com/facebook/react/defaults/DefaultNewArchitectureEntryPoint.kt

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,6 @@ public object DefaultNewArchitectureEntryPoint {
102102
}
103103
}
104104

105-
privateFabricEnabled = fabricEnabled
106105
privateTurboModulesEnabled = turboModulesEnabled
107106
privateConcurrentReactEnabled = fabricEnabled
108107
privateBridgelessEnabled = bridgelessEnabled
@@ -120,7 +119,7 @@ public object DefaultNewArchitectureEntryPoint {
120119
val (isValid, errorMessage) =
121120
isConfigurationValid(
122121
privateTurboModulesEnabled,
123-
privateFabricEnabled,
122+
true,
124123
privateBridgelessEnabled,
125124
)
126125
if (!isValid) {
@@ -130,8 +129,6 @@ public object DefaultNewArchitectureEntryPoint {
130129
DefaultSoLoader.maybeLoadSoLibrary()
131130
}
132131

133-
private var privateFabricEnabled: Boolean = false
134-
135132
@JvmStatic
136133
public val fabricEnabled: Boolean
137134
get() = true

0 commit comments

Comments
 (0)