Skip to content

Commit ae85be3

Browse files
sammy-SCfacebook-github-bot
authored andcommitted
remove const from UIManager::createNode and UIManager::cloneNode
Summary: changelog: [internal] These shadow nodes are freshly created and unsealed. Return non-const. Reviewed By: NickGerleman Differential Revision: D50796024 fbshipit-source-id: 6e2a61bb03efbc6f63a489928787e645971780df
1 parent 475b835 commit ae85be3

File tree

4 files changed

+6
-6
lines changed

4 files changed

+6
-6
lines changed

packages/react-native/ReactCommon/react/renderer/core/ComponentDescriptor.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ class ComponentDescriptor {
7878
/*
7979
* Creates a new `ShadowNode` of a particular component type.
8080
*/
81-
virtual ShadowNode::Shared createShadowNode(
81+
virtual std::shared_ptr<ShadowNode> createShadowNode(
8282
const ShadowNodeFragment& fragment,
8383
const ShadowNodeFamily::Shared& family) const = 0;
8484

packages/react-native/ReactCommon/react/renderer/core/ConcreteComponentDescriptor.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@ class ConcreteComponentDescriptor : public ComponentDescriptor {
6363
return ShadowNodeT::BaseTraits();
6464
}
6565

66-
ShadowNode::Shared createShadowNode(
66+
std::shared_ptr<ShadowNode> createShadowNode(
6767
const ShadowNodeFragment& fragment,
6868
const ShadowNodeFamily::Shared& family) const override {
6969
auto shadowNode =

packages/react-native/ReactCommon/react/renderer/uimanager/UIManager.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@ UIManager::~UIManager() {
6262
<< ").";
6363
}
6464

65-
ShadowNode::Shared UIManager::createNode(
65+
std::shared_ptr<ShadowNode> UIManager::createNode(
6666
Tag tag,
6767
const std::string& name,
6868
SurfaceId surfaceId,
@@ -109,7 +109,7 @@ ShadowNode::Shared UIManager::createNode(
109109
return shadowNode;
110110
}
111111

112-
ShadowNode::Shared UIManager::cloneNode(
112+
std::shared_ptr<ShadowNode> UIManager::cloneNode(
113113
const ShadowNode& shadowNode,
114114
const ShadowNode::SharedListOfShared& children,
115115
const RawProps* rawProps) const {

packages/react-native/ReactCommon/react/renderer/uimanager/UIManager.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -130,14 +130,14 @@ class UIManager final : public ShadowTreeDelegate {
130130
const RootShadowNode::Shared& oldRootShadowNode,
131131
const RootShadowNode::Unshared& newRootShadowNode) const override;
132132

133-
ShadowNode::Shared createNode(
133+
std::shared_ptr<ShadowNode> createNode(
134134
Tag tag,
135135
const std::string& componentName,
136136
SurfaceId surfaceId,
137137
const RawProps& props,
138138
const InstanceHandle::Shared& instanceHandle) const;
139139

140-
ShadowNode::Shared cloneNode(
140+
std::shared_ptr<ShadowNode> cloneNode(
141141
const ShadowNode& shadowNode,
142142
const ShadowNode::SharedListOfShared& children = nullptr,
143143
const RawProps* rawProps = nullptr) const;

0 commit comments

Comments
 (0)