Skip to content

Commit 2c06985

Browse files
Use correct base class for NativeReactNativeFeatureFlags (#54046)
Summary: Changelog: [Internal] Per documentation the base class to extend from is the non `JSI` one: https://reactnative.dev/docs/the-new-architecture/pure-cxx-modules#3-write-the-native-code Reviewed By: rubennorte Differential Revision: D83815509
1 parent 36e1c0a commit 2c06985

File tree

4 files changed

+6
-6
lines changed

4 files changed

+6
-6
lines changed

packages/react-native/ReactCommon/react/nativemodule/featureflags/NativeReactNativeFeatureFlags.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
* This source code is licensed under the MIT license found in the
55
* LICENSE file in the root directory of this source tree.
66
*
7-
* @generated SignedSource<<e799e5ce5aea7adbef02a7e7431f8a35>>
7+
* @generated SignedSource<<4d5d0795ed4cea8c1e185f6fe3fa0fad>>
88
*/
99

1010
/**
@@ -35,7 +35,7 @@ namespace facebook::react {
3535

3636
NativeReactNativeFeatureFlags::NativeReactNativeFeatureFlags(
3737
std::shared_ptr<CallInvoker> jsInvoker)
38-
: NativeReactNativeFeatureFlagsCxxSpecJSI(std::move(jsInvoker)) {}
38+
: NativeReactNativeFeatureFlagsCxxSpec(std::move(jsInvoker)) {}
3939

4040
bool NativeReactNativeFeatureFlags::commonTestFlag(
4141
jsi::Runtime& /*runtime*/) {

packages/react-native/ReactCommon/react/nativemodule/featureflags/NativeReactNativeFeatureFlags.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
* This source code is licensed under the MIT license found in the
55
* LICENSE file in the root directory of this source tree.
66
*
7-
* @generated SignedSource<<990b6518e7b1d196deb0965e9ada6605>>
7+
* @generated SignedSource<<c85e22d73a1e272c7838c4a4e9db9b6d>>
88
*/
99

1010
/**
@@ -28,7 +28,7 @@
2828
namespace facebook::react {
2929

3030
class NativeReactNativeFeatureFlags
31-
: public NativeReactNativeFeatureFlagsCxxSpecJSI {
31+
: public NativeReactNativeFeatureFlagsCxxSpec<NativeReactNativeFeatureFlags> {
3232
public:
3333
NativeReactNativeFeatureFlags(std::shared_ptr<CallInvoker> jsInvoker);
3434

packages/react-native/scripts/featureflags/templates/js/NativeReactNativeFeatureFlags.cpp-template.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ namespace facebook::react {
4747
4848
NativeReactNativeFeatureFlags::NativeReactNativeFeatureFlags(
4949
std::shared_ptr<CallInvoker> jsInvoker)
50-
: NativeReactNativeFeatureFlagsCxxSpecJSI(std::move(jsInvoker)) {}
50+
: NativeReactNativeFeatureFlagsCxxSpec(std::move(jsInvoker)) {}
5151
5252
${Object.entries(definitions.common)
5353
.map(([flagName, flagConfig]) =>

packages/react-native/scripts/featureflags/templates/js/NativeReactNativeFeatureFlags.h-template.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ ${DO_NOT_MODIFY_COMMENT}
3636
namespace facebook::react {
3737
3838
class NativeReactNativeFeatureFlags
39-
: public NativeReactNativeFeatureFlagsCxxSpecJSI {
39+
: public NativeReactNativeFeatureFlagsCxxSpec<NativeReactNativeFeatureFlags> {
4040
public:
4141
NativeReactNativeFeatureFlags(std::shared_ptr<CallInvoker> jsInvoker);
4242

0 commit comments

Comments
 (0)