Skip to content

Commit b539a7e

Browse files
sammy-SCfacebook-github-bot
authored andcommitted
remove return value from synchronouslyUpdateViewOnUIThread
Differential Revision: D74884875
1 parent ba6323b commit b539a7e

File tree

3 files changed

+6
-7
lines changed

3 files changed

+6
-7
lines changed

packages/react-native/React/Fabric/RCTSurfacePresenter.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,7 @@ NS_ASSUME_NONNULL_BEGIN
6565

6666
- (nullable RCTFabricSurface *)surfaceForRootTag:(ReactTag)rootTag;
6767

68-
- (BOOL)synchronouslyUpdateViewOnUIThread:(NSNumber *)reactTag props:(NSDictionary *)props;
68+
- (void)synchronouslyUpdateViewOnUIThread:(NSNumber *)reactTag props:(NSDictionary *)props;
6969

7070
- (void)setupAnimationDriverWithSurfaceHandler:(const facebook::react::SurfaceHandler &)surfaceHandler;
7171

packages/react-native/React/Fabric/RCTSurfacePresenter.mm

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -148,28 +148,27 @@ - (UIView *)findComponentViewWithTag_DO_NOT_USE_DEPRECATED:(NSInteger)tag
148148
return componentView;
149149
}
150150

151-
- (BOOL)synchronouslyUpdateViewOnUIThread:(NSNumber *)reactTag props:(NSDictionary *)props
151+
- (void)synchronouslyUpdateViewOnUIThread:(NSNumber *)reactTag props:(NSDictionary *)props
152152
{
153153
RCTScheduler *scheduler = [self scheduler];
154154
if (!scheduler) {
155-
return NO;
155+
return;
156156
}
157157

158158
ReactTag tag = [reactTag integerValue];
159159
UIView<RCTComponentViewProtocol> *componentView =
160160
[_mountingManager.componentViewRegistry findComponentViewWithTag:tag];
161161
if (componentView == nil) {
162-
return NO; // This view probably isn't managed by Fabric
162+
return; // This view probably isn't managed by Fabric
163163
}
164164
ComponentHandle handle = [[componentView class] componentDescriptorProvider].handle;
165165
auto *componentDescriptor = [scheduler findComponentDescriptorByHandle_DO_NOT_USE_THIS_IS_BROKEN:handle];
166166

167167
if (!componentDescriptor) {
168-
return YES;
168+
return;
169169
}
170170

171171
[_mountingManager synchronouslyUpdateViewOnUIThread:tag changedProps:props componentDescriptor:*componentDescriptor];
172-
return YES;
173172
}
174173

175174
- (void)setupAnimationDriverWithSurfaceHandler:(const facebook::react::SurfaceHandler &)surfaceHandler

packages/react-native/React/Modules/RCTSurfacePresenterStub.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ NS_ASSUME_NONNULL_BEGIN
3232
- (id<RCTSurfaceProtocol>)createFabricSurfaceForModuleName:(NSString *)moduleName
3333
initialProperties:(NSDictionary *)initialProperties;
3434
- (nullable UIView *)findComponentViewWithTag_DO_NOT_USE_DEPRECATED:(NSInteger)tag;
35-
- (BOOL)synchronouslyUpdateViewOnUIThread:(NSNumber *)reactTag props:(NSDictionary *)props;
35+
- (void)synchronouslyUpdateViewOnUIThread:(NSNumber *)reactTag props:(NSDictionary *)props;
3636
- (void)addObserver:(id<RCTSurfacePresenterObserver>)observer;
3737
- (void)removeObserver:(id<RCTSurfacePresenterObserver>)observer;
3838

0 commit comments

Comments
 (0)