From ae85be3e923fd6f59ca9ff1c9dcb5de5c879bb46 Mon Sep 17 00:00:00 2001 From: Samuel Susla Date: Tue, 31 Oct 2023 09:58:05 -0700 Subject: [PATCH] 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 --- .../ReactCommon/react/renderer/core/ComponentDescriptor.h | 2 +- .../react/renderer/core/ConcreteComponentDescriptor.h | 2 +- .../ReactCommon/react/renderer/uimanager/UIManager.cpp | 4 ++-- .../ReactCommon/react/renderer/uimanager/UIManager.h | 4 ++-- 4 files changed, 6 insertions(+), 6 deletions(-) diff --git a/packages/react-native/ReactCommon/react/renderer/core/ComponentDescriptor.h b/packages/react-native/ReactCommon/react/renderer/core/ComponentDescriptor.h index 59f1840833b..aa9c40fc8dd 100644 --- a/packages/react-native/ReactCommon/react/renderer/core/ComponentDescriptor.h +++ b/packages/react-native/ReactCommon/react/renderer/core/ComponentDescriptor.h @@ -78,7 +78,7 @@ class ComponentDescriptor { /* * Creates a new `ShadowNode` of a particular component type. */ - virtual ShadowNode::Shared createShadowNode( + virtual std::shared_ptr createShadowNode( const ShadowNodeFragment& fragment, const ShadowNodeFamily::Shared& family) const = 0; diff --git a/packages/react-native/ReactCommon/react/renderer/core/ConcreteComponentDescriptor.h b/packages/react-native/ReactCommon/react/renderer/core/ConcreteComponentDescriptor.h index a57c3b1318f..6f8836cab45 100644 --- a/packages/react-native/ReactCommon/react/renderer/core/ConcreteComponentDescriptor.h +++ b/packages/react-native/ReactCommon/react/renderer/core/ConcreteComponentDescriptor.h @@ -63,7 +63,7 @@ class ConcreteComponentDescriptor : public ComponentDescriptor { return ShadowNodeT::BaseTraits(); } - ShadowNode::Shared createShadowNode( + std::shared_ptr createShadowNode( const ShadowNodeFragment& fragment, const ShadowNodeFamily::Shared& family) const override { auto shadowNode = diff --git a/packages/react-native/ReactCommon/react/renderer/uimanager/UIManager.cpp b/packages/react-native/ReactCommon/react/renderer/uimanager/UIManager.cpp index 135dc5516a9..e16fff45294 100644 --- a/packages/react-native/ReactCommon/react/renderer/uimanager/UIManager.cpp +++ b/packages/react-native/ReactCommon/react/renderer/uimanager/UIManager.cpp @@ -62,7 +62,7 @@ UIManager::~UIManager() { << ")."; } -ShadowNode::Shared UIManager::createNode( +std::shared_ptr UIManager::createNode( Tag tag, const std::string& name, SurfaceId surfaceId, @@ -109,7 +109,7 @@ ShadowNode::Shared UIManager::createNode( return shadowNode; } -ShadowNode::Shared UIManager::cloneNode( +std::shared_ptr UIManager::cloneNode( const ShadowNode& shadowNode, const ShadowNode::SharedListOfShared& children, const RawProps* rawProps) const { diff --git a/packages/react-native/ReactCommon/react/renderer/uimanager/UIManager.h b/packages/react-native/ReactCommon/react/renderer/uimanager/UIManager.h index 04baf7f7bb9..3f2abc2bb82 100644 --- a/packages/react-native/ReactCommon/react/renderer/uimanager/UIManager.h +++ b/packages/react-native/ReactCommon/react/renderer/uimanager/UIManager.h @@ -130,14 +130,14 @@ class UIManager final : public ShadowTreeDelegate { const RootShadowNode::Shared& oldRootShadowNode, const RootShadowNode::Unshared& newRootShadowNode) const override; - ShadowNode::Shared createNode( + std::shared_ptr createNode( Tag tag, const std::string& componentName, SurfaceId surfaceId, const RawProps& props, const InstanceHandle::Shared& instanceHandle) const; - ShadowNode::Shared cloneNode( + std::shared_ptr cloneNode( const ShadowNode& shadowNode, const ShadowNode::SharedListOfShared& children = nullptr, const RawProps* rawProps = nullptr) const;