Remove config variant copy ctor from YGNode (#37091)

Summary:
X-link: https://github.com/facebook/react-native/pull/37091

Pull Request resolved: https://github.com/facebook/yoga/pull/1258

This private constructor was added specifically for Fabric when config setting was deprecated, but that is undeprecated now. Fbsource fabric was moved off of it, and the RN desktop for was in the last diff in the stack, so we can remove it now.

Reviewed By: yungsters

Differential Revision: D45292729

fbshipit-source-id: 87b2a1adaafaf817befe44dbc3ac178af59a6e68
This commit is contained in:
Nick Gerleman
2023-04-27 06:48:04 -07:00
committed by Facebook GitHub Bot
parent c878ceeabd
commit 36406ce17c
2 changed files with 0 additions and 15 deletions

View File

@@ -23,17 +23,6 @@ YGNode::YGNode(const YGConfigRef config) : config_{config} {
}
};
YGNode::YGNode(const YGNode& node, YGConfigRef config) : YGNode{node} {
YGAssert(
config != nullptr, "Attempting to construct YGNode with null config");
config_ = config;
flags_.hasNewLayout = true;
if (config->useWebDefaults) {
useWebDefaults();
}
}
YGNode::YGNode(YGNode&& node) {
context_ = node.context_;
flags_ = node.flags_;

View File

@@ -93,10 +93,6 @@ public:
// Should we remove this?
YGNode(const YGNode& node) = default;
[[deprecated("Will be removed imminently")]] YGNode(
const YGNode& node,
YGConfigRef config);
// assignment means potential leaks of existing children, or alternatively
// freeing unowned memory, double free, or freeing stack memory.
YGNode& operator=(const YGNode&) = delete;