Skip to content

Commit 2148f15

Browse files
christophpurrermeta-codesync[bot]
authored andcommitted
RNTester / RCTReactNativeFactory remove unused new architecture flags and RCTArchConfiguratorProtocol (#57054)
Summary: Pull Request resolved: #57054 ## Changelog: [iOS][Breaking] RNTester / RCTReactNativeFactory remove unused new architecture flags and RCTArchConfiguratorProtocol Values are already `true`/YES ``` [[RCTRootViewFactoryConfiguration alloc] initWithBundleURLBlock:bundleUrlBlock newArchEnabled:YES turboModuleEnabled:YES bridgelessEnabled:YES]; ``` Reviewed By: cipolleschi Differential Revision: D107305248 fbshipit-source-id: bd09d12cf1d22ec6bb435333c05004b73fce98c5
1 parent b766986 commit 2148f15

8 files changed

Lines changed: 4 additions & 95 deletions

File tree

packages/react-native/Libraries/AppDelegate/RCTArchConfiguratorProtocol.h

Lines changed: 0 additions & 38 deletions
This file was deleted.

packages/react-native/Libraries/AppDelegate/RCTReactNativeFactory.h

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@
88
#import <React/RCTBridgeDelegate.h>
99
#import <React/RCTConvert.h>
1010
#import <UIKit/UIKit.h>
11-
#import "RCTArchConfiguratorProtocol.h"
1211
#import "RCTDependencyProvider.h"
1312
#import "RCTJSRuntimeConfiguratorProtocol.h"
1413
#import "RCTRootViewFactory.h"
@@ -39,8 +38,7 @@ typedef NS_ENUM(NSInteger, RCTReleaseLevel) { Canary, Experimental, Stable };
3938
RCTTurboModuleManagerDelegate,
4039
RCTComponentViewFactoryComponentProvider,
4140
#endif
42-
RCTJSRuntimeConfiguratorProtocol,
43-
RCTArchConfiguratorProtocol>
41+
RCTJSRuntimeConfiguratorProtocol>
4442

4543
/// Return the bundle URL for the main bundle.
4644
- (NSURL *__nullable)bundleURL;

packages/react-native/Libraries/AppDelegate/RCTReactNativeFactory.mm

Lines changed: 0 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -132,28 +132,6 @@ - (JSRuntimeFactoryRef)createJSRuntimeFactory
132132
return [_delegate createJSRuntimeFactory];
133133
}
134134

135-
#pragma mark - RCTArchConfiguratorProtocol
136-
137-
- (BOOL)newArchEnabled
138-
{
139-
return YES;
140-
}
141-
142-
- (BOOL)fabricEnabled
143-
{
144-
return YES;
145-
}
146-
147-
- (BOOL)turboModuleEnabled
148-
{
149-
return YES;
150-
}
151-
152-
- (BOOL)bridgelessEnabled
153-
{
154-
return YES;
155-
}
156-
157135
#pragma mark - RCTTurboModuleManagerDelegate
158136

159137
- (Class)getModuleClassFromName:(const char *)name

packages/react-native/scripts/ios-prebuild/templates/React_RCTAppDelegate-umbrella.h

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,6 @@
1919

2020
#import "RCTAppDelegate.h"
2121
#import "RCTAppSetupUtils.h"
22-
#import "RCTArchConfiguratorProtocol.h"
2322
#import "RCTDefaultReactNativeFactoryDelegate.h"
2423
#import "RCTDependencyProvider.h"
2524
#import "RCTJSRuntimeConfiguratorProtocol.h"

packages/rn-tester/RNTester/AppDelegate.mm

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -159,13 +159,6 @@ - (void)userNotificationCenter:(UNUserNotificationCenter *)center
159159
}
160160
#endif
161161

162-
#pragma mark - New Arch Enabled settings
163-
164-
- (BOOL)bridgelessEnabled
165-
{
166-
return YES;
167-
}
168-
169162
#pragma mark - RCTComponentViewFactoryComponentProvider
170163

171164
#ifndef RN_DISABLE_OSS_PLUGIN_HEADER

scripts/cxx-api/api-snapshots/ReactAppleDebugCxx.api

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -2608,13 +2608,6 @@ protocol RCTAnimationDriver : public NSObject {
26082608
public virtual void stopAnimation();
26092609
}
26102610

2611-
protocol RCTArchConfiguratorProtocol {
2612-
public virtual BOOL bridgelessEnabled();
2613-
public virtual BOOL fabricEnabled();
2614-
public virtual BOOL newArchEnabled();
2615-
public virtual BOOL turboModuleEnabled();
2616-
}
2617-
26182611
protocol RCTAutoInsetsProtocol {
26192612
public @property (assign) BOOL automaticallyAdjustContentInsets;
26202613
public @property (assign) UIEdgeInsets contentInset;
@@ -2985,7 +2978,7 @@ protocol RCTPullToRefreshViewViewProtocol : public NSObject {
29852978
public virtual void setNativeRefreshing:(BOOL refreshing);
29862979
}
29872980

2988-
protocol RCTReactNativeFactoryDelegate : public RCTBridgeDelegate, public RCTUIConfiguratorProtocol, public RCTHostDelegate, public RCTTurboModuleManagerDelegate, public RCTComponentViewFactoryComponentProvider, public RCTJSRuntimeConfiguratorProtocol, public RCTArchConfiguratorProtocol {
2981+
protocol RCTReactNativeFactoryDelegate : public RCTBridgeDelegate, public RCTUIConfiguratorProtocol, public RCTHostDelegate, public RCTTurboModuleManagerDelegate, public RCTComponentViewFactoryComponentProvider, public RCTJSRuntimeConfiguratorProtocol {
29892982
public @property (strong) id<RCTDependencyProvider> dependencyProvider;
29902983
public virtual NSDictionary<NSString*, Class<RCTComponentViewProtocol>>* thirdPartyFabricComponents();
29912984
public virtual NSURL* _Nullable bundleURL();

scripts/cxx-api/api-snapshots/ReactAppleNewarchCxx.api

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -2596,13 +2596,6 @@ protocol RCTAnimationDriver : public NSObject {
25962596
public virtual void stopAnimation();
25972597
}
25982598

2599-
protocol RCTArchConfiguratorProtocol {
2600-
public virtual BOOL bridgelessEnabled();
2601-
public virtual BOOL fabricEnabled();
2602-
public virtual BOOL newArchEnabled();
2603-
public virtual BOOL turboModuleEnabled();
2604-
}
2605-
26062599
protocol RCTAutoInsetsProtocol {
26072600
public @property (assign) BOOL automaticallyAdjustContentInsets;
26082601
public @property (assign) UIEdgeInsets contentInset;
@@ -2972,7 +2965,7 @@ protocol RCTPullToRefreshViewViewProtocol : public NSObject {
29722965
public virtual void setNativeRefreshing:(BOOL refreshing);
29732966
}
29742967

2975-
protocol RCTReactNativeFactoryDelegate : public RCTBridgeDelegate, public RCTUIConfiguratorProtocol, public RCTHostDelegate, public RCTTurboModuleManagerDelegate, public RCTComponentViewFactoryComponentProvider, public RCTJSRuntimeConfiguratorProtocol, public RCTArchConfiguratorProtocol {
2968+
protocol RCTReactNativeFactoryDelegate : public RCTBridgeDelegate, public RCTUIConfiguratorProtocol, public RCTHostDelegate, public RCTTurboModuleManagerDelegate, public RCTComponentViewFactoryComponentProvider, public RCTJSRuntimeConfiguratorProtocol {
29762969
public @property (strong) id<RCTDependencyProvider> dependencyProvider;
29772970
public virtual NSDictionary<NSString*, Class<RCTComponentViewProtocol>>* thirdPartyFabricComponents();
29782971
public virtual NSURL* _Nullable bundleURL();

scripts/cxx-api/api-snapshots/ReactAppleReleaseCxx.api

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -2608,13 +2608,6 @@ protocol RCTAnimationDriver : public NSObject {
26082608
public virtual void stopAnimation();
26092609
}
26102610

2611-
protocol RCTArchConfiguratorProtocol {
2612-
public virtual BOOL bridgelessEnabled();
2613-
public virtual BOOL fabricEnabled();
2614-
public virtual BOOL newArchEnabled();
2615-
public virtual BOOL turboModuleEnabled();
2616-
}
2617-
26182611
protocol RCTAutoInsetsProtocol {
26192612
public @property (assign) BOOL automaticallyAdjustContentInsets;
26202613
public @property (assign) UIEdgeInsets contentInset;
@@ -2985,7 +2978,7 @@ protocol RCTPullToRefreshViewViewProtocol : public NSObject {
29852978
public virtual void setNativeRefreshing:(BOOL refreshing);
29862979
}
29872980

2988-
protocol RCTReactNativeFactoryDelegate : public RCTBridgeDelegate, public RCTUIConfiguratorProtocol, public RCTHostDelegate, public RCTTurboModuleManagerDelegate, public RCTComponentViewFactoryComponentProvider, public RCTJSRuntimeConfiguratorProtocol, public RCTArchConfiguratorProtocol {
2981+
protocol RCTReactNativeFactoryDelegate : public RCTBridgeDelegate, public RCTUIConfiguratorProtocol, public RCTHostDelegate, public RCTTurboModuleManagerDelegate, public RCTComponentViewFactoryComponentProvider, public RCTJSRuntimeConfiguratorProtocol {
29892982
public @property (strong) id<RCTDependencyProvider> dependencyProvider;
29902983
public virtual NSDictionary<NSString*, Class<RCTComponentViewProtocol>>* thirdPartyFabricComponents();
29912984
public virtual NSURL* _Nullable bundleURL();

0 commit comments

Comments
 (0)