From 67b59b6f0a612005dd2b1b00cde9b8027b656ae5 Mon Sep 17 00:00:00 2001 From: Samuel Susla Date: Tue, 24 Feb 2026 14:37:08 -0800 Subject: [PATCH] Remove deprecated ShadowNode type aliases (#55687) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Summary: ## Changelog: [General] [Breaking]: Removing deprecated type aliases. Use the type directly. Remove deprecated type aliases from ShadowNode that were scheduled for deletion in 0.84: - `ShadowNode::Shared` → use `std::shared_ptr` - `ShadowNode::Weak` → use `std::weak_ptr` - `ShadowNode::Unshared` → use `std::shared_ptr` - `ShadowNode::ListOfShared` → use `std::vector>` - `ShadowNode::ListOfWeak` → use `std::vector>` Reviewed By: lenaic Differential Revision: D90331085 --- .../ReactCommon/react/renderer/core/ShadowNode.h | 12 ------------ .../react/test_utils/shadowTreeGeneration.h | 13 ++++++++----- 2 files changed, 8 insertions(+), 17 deletions(-) diff --git a/packages/react-native/ReactCommon/react/renderer/core/ShadowNode.h b/packages/react-native/ReactCommon/react/renderer/core/ShadowNode.h index d4517e529ff5..6a0bf6102691 100644 --- a/packages/react-native/ReactCommon/react/renderer/core/ShadowNode.h +++ b/packages/react-native/ReactCommon/react/renderer/core/ShadowNode.h @@ -30,18 +30,6 @@ struct ShadowNodeWrapper; class ShadowNode : public Sealable, public DebugStringConvertible, public jsi::NativeState { public: - // TODO(T223558094): delete this in the next version. - using Shared [[deprecated("Use std::shared_ptr instead")]] = std::shared_ptr; - // TODO(T223558094): delete this in the next version. - using Weak [[deprecated("Use std::weak_ptr instead")]] = std::weak_ptr; - // TODO(T223558094): delete this in the next version. - using Unshared [[deprecated("Use std::shared_ptr instead")]] = std::shared_ptr; - // TODO(T223558094): delete this in the next version. - using ListOfShared [[deprecated("Use std::vector> instead")]] = - std::vector>; - // TODO(T223558094): delete this in the next version. - using ListOfWeak [[deprecated("Use std::vector> instead")]] = - std::vector>; using SharedListOfShared = std::shared_ptr>>; using UnsharedListOfShared = std::shared_ptr>>; using UnsharedListOfWeak = std::shared_ptr>>; diff --git a/packages/react-native/ReactCommon/react/test_utils/shadowTreeGeneration.h b/packages/react-native/ReactCommon/react/test_utils/shadowTreeGeneration.h index 95390ef3ca8d..6627592789c9 100644 --- a/packages/react-native/ReactCommon/react/test_utils/shadowTreeGeneration.h +++ b/packages/react-native/ReactCommon/react/test_utils/shadowTreeGeneration.h @@ -89,9 +89,10 @@ static ShadowTreeEdge findRandomShadowNode( return findShadowNodeWithIndex(rootShadowNode, entropy.random(1 /* Excluding a root node */, count - 1)); } -static ShadowNode::ListOfShared cloneSharedShadowNodeList(const ShadowNode::ListOfShared &list) +static std::vector> cloneSharedShadowNodeList( + const std::vector> &list) { - auto result = ShadowNode::ListOfShared{}; + auto result = std::vector>{}; result.reserve(list.size()); for (const auto &shadowNode : list) { result.push_back(shadowNode->clone({})); @@ -105,7 +106,8 @@ static inline std::shared_ptr messWithChildren(const Entropy &entrop children = cloneSharedShadowNodeList(children); entropy.shuffle(children); return shadowNode.clone( - {ShadowNodeFragment::propsPlaceholder(), std::make_shared(children)}); + {ShadowNodeFragment::propsPlaceholder(), + std::make_shared>>(children)}); } static inline std::shared_ptr messWithLayoutableOnlyFlag( @@ -280,7 +282,7 @@ static inline std::shared_ptr generateShadowNodeTree( auto items = std::vector(size); std::fill(items.begin(), items.end(), 1); auto chunks = entropy.distribute(items, deviation); - auto children = ShadowNode::ListOfShared{}; + auto children = std::vector>{}; for (const auto &chunk : chunks) { children.push_back(generateShadowNodeTree(entropy, componentDescriptor, chunk.size())); @@ -289,7 +291,8 @@ static inline std::shared_ptr generateShadowNodeTree( auto family = componentDescriptor.createFamily({generateReactTag(), SurfaceId(1), nullptr}); return componentDescriptor.createShadowNode( ShadowNodeFragment{ - generateDefaultProps(componentDescriptor), std::make_shared(children)}, + generateDefaultProps(componentDescriptor), + std::make_shared>>(children)}, family); }