Skip to content

Comments

Deprecate and replace obscure ShadowNode type aliases#55690

Open
sammy-SC wants to merge 2 commits intofacebook:mainfrom
sammy-SC:export-D90331677
Open

Deprecate and replace obscure ShadowNode type aliases#55690
sammy-SC wants to merge 2 commits intofacebook:mainfrom
sammy-SC:export-D90331677

Conversation

@sammy-SC
Copy link
Contributor

Summary:
Changelog: [internal]

Mark these obscure type aliases as deprecated and replace all usages with explicit types:

  • ShadowNode::SharedListOfSharedstd::shared_ptr<const std::vector<std::shared_ptr<const ShadowNode>>>
  • ShadowNode::UnsharedListOfSharedstd::shared_ptr<std::vector<std::shared_ptr<const ShadowNode>>>
  • ShadowNode::UnsharedListOfWeakstd::shared_ptr<std::vector<std::weak_ptr<const ShadowNode>>>

the type aliases will be deleted later.

Reviewed By: lenaic

Differential Revision: D90331677

@meta-cla meta-cla bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Feb 23, 2026
@meta-codesync
Copy link

meta-codesync bot commented Feb 23, 2026

@sammy-SC has exported this pull request. If you are a Meta employee, you can view the originating Diff in D90331677.

sammy-SC added a commit to sammy-SC/react-native that referenced this pull request Feb 23, 2026
Summary:

Changelog: [internal]

Mark these obscure type aliases as deprecated and replace all usages with explicit types:
- `ShadowNode::SharedListOfShared` → `std::shared_ptr<const std::vector<std::shared_ptr<const ShadowNode>>>`
- `ShadowNode::UnsharedListOfShared` → `std::shared_ptr<std::vector<std::shared_ptr<const ShadowNode>>>`
- `ShadowNode::UnsharedListOfWeak` → `std::shared_ptr<std::vector<std::weak_ptr<const ShadowNode>>>`

the type aliases will be deleted later.

Reviewed By: lenaic

Differential Revision: D90331677
sammy-SC added a commit to sammy-SC/react-native that referenced this pull request Feb 23, 2026
Summary:

Changelog: [internal]

Mark these obscure type aliases as deprecated and replace all usages with explicit types:
- `ShadowNode::SharedListOfShared` → `std::shared_ptr<const std::vector<std::shared_ptr<const ShadowNode>>>`
- `ShadowNode::UnsharedListOfShared` → `std::shared_ptr<std::vector<std::shared_ptr<const ShadowNode>>>`
- `ShadowNode::UnsharedListOfWeak` → `std::shared_ptr<std::vector<std::weak_ptr<const ShadowNode>>>`

the type aliases will be deleted later.

Reviewed By: lenaic

Differential Revision: D90331677
sammy-SC added a commit to sammy-SC/react-native that referenced this pull request Feb 23, 2026
Summary:

Changelog: [internal]

Mark these obscure type aliases as deprecated and replace all usages with explicit types:
- `ShadowNode::SharedListOfShared` → `std::shared_ptr<const std::vector<std::shared_ptr<const ShadowNode>>>`
- `ShadowNode::UnsharedListOfShared` → `std::shared_ptr<std::vector<std::shared_ptr<const ShadowNode>>>`
- `ShadowNode::UnsharedListOfWeak` → `std::shared_ptr<std::vector<std::weak_ptr<const ShadowNode>>>`

the type aliases will be deleted later.

Reviewed By: lenaic

Differential Revision: D90331677
sammy-SC added a commit to sammy-SC/react-native that referenced this pull request Feb 24, 2026
Summary:

Changelog: [internal]

Mark these obscure type aliases as deprecated and replace all usages with explicit types:
- `ShadowNode::SharedListOfShared` → `std::shared_ptr<const std::vector<std::shared_ptr<const ShadowNode>>>`
- `ShadowNode::UnsharedListOfShared` → `std::shared_ptr<std::vector<std::shared_ptr<const ShadowNode>>>`
- `ShadowNode::UnsharedListOfWeak` → `std::shared_ptr<std::vector<std::weak_ptr<const ShadowNode>>>`

the type aliases will be deleted later.

Reviewed By: lenaic

Differential Revision: D90331677
sammy-SC added a commit to sammy-SC/react-native that referenced this pull request Feb 24, 2026
Summary:

Changelog: [internal]

Mark these obscure type aliases as deprecated and replace all usages with explicit types:
- `ShadowNode::SharedListOfShared` → `std::shared_ptr<const std::vector<std::shared_ptr<const ShadowNode>>>`
- `ShadowNode::UnsharedListOfShared` → `std::shared_ptr<std::vector<std::shared_ptr<const ShadowNode>>>`
- `ShadowNode::UnsharedListOfWeak` → `std::shared_ptr<std::vector<std::weak_ptr<const ShadowNode>>>`

the type aliases will be deleted later.

Reviewed By: lenaic

Differential Revision: D90331677
sammy-SC added a commit to sammy-SC/react-native that referenced this pull request Feb 24, 2026
Summary:

Changelog: [internal]

Mark these obscure type aliases as deprecated and replace all usages with explicit types:
- `ShadowNode::SharedListOfShared` → `std::shared_ptr<const std::vector<std::shared_ptr<const ShadowNode>>>`
- `ShadowNode::UnsharedListOfShared` → `std::shared_ptr<std::vector<std::shared_ptr<const ShadowNode>>>`
- `ShadowNode::UnsharedListOfWeak` → `std::shared_ptr<std::vector<std::weak_ptr<const ShadowNode>>>`

the type aliases will be deleted later.

Reviewed By: lenaic

Differential Revision: D90331677
sammy-SC added a commit to sammy-SC/react-native that referenced this pull request Feb 24, 2026
Summary:

Changelog: [internal]

Mark these obscure type aliases as deprecated and replace all usages with explicit types:
- `ShadowNode::SharedListOfShared` → `std::shared_ptr<const std::vector<std::shared_ptr<const ShadowNode>>>`
- `ShadowNode::UnsharedListOfShared` → `std::shared_ptr<std::vector<std::shared_ptr<const ShadowNode>>>`
- `ShadowNode::UnsharedListOfWeak` → `std::shared_ptr<std::vector<std::weak_ptr<const ShadowNode>>>`

the type aliases will be deleted later.

Reviewed By: lenaic

Differential Revision: D90331677
sammy-SC added a commit to sammy-SC/react-native that referenced this pull request Feb 24, 2026
Summary:

Changelog: [internal]

Mark these obscure type aliases as deprecated and replace all usages with explicit types:
- `ShadowNode::SharedListOfShared` → `std::shared_ptr<const std::vector<std::shared_ptr<const ShadowNode>>>`
- `ShadowNode::UnsharedListOfShared` → `std::shared_ptr<std::vector<std::shared_ptr<const ShadowNode>>>`
- `ShadowNode::UnsharedListOfWeak` → `std::shared_ptr<std::vector<std::weak_ptr<const ShadowNode>>>`

the type aliases will be deleted later.

Reviewed By: lenaic

Differential Revision: D90331677
Summary:
Remove deprecated type aliases that are no longer used:
- `SharedImageManager` → use `std::shared_ptr<ImageManager>`
- `ContextContainer::Shared` → use `std::shared_ptr<const ContextContainer>`

These were previously removed, brought back as deprecated for compatibility, and now have no remaining usages.

Differential Revision: D90331219
sammy-SC added a commit to sammy-SC/react-native that referenced this pull request Feb 25, 2026
Summary:

Changelog: [internal]

Mark these obscure type aliases as deprecated and replace all usages with explicit types:
- `ShadowNode::SharedListOfShared` → `std::shared_ptr<const std::vector<std::shared_ptr<const ShadowNode>>>`
- `ShadowNode::UnsharedListOfShared` → `std::shared_ptr<std::vector<std::shared_ptr<const ShadowNode>>>`
- `ShadowNode::UnsharedListOfWeak` → `std::shared_ptr<std::vector<std::weak_ptr<const ShadowNode>>>`

the type aliases will be deleted later.

Reviewed By: lenaic

Differential Revision: D90331677
Summary:
Pull Request resolved: facebook#55690

Changelog: [internal]

Mark these obscure type aliases as deprecated and replace all usages with explicit types:
- `ShadowNode::SharedListOfShared` → `std::shared_ptr<const std::vector<std::shared_ptr<const ShadowNode>>>`
- `ShadowNode::UnsharedListOfShared` → `std::shared_ptr<std::vector<std::shared_ptr<const ShadowNode>>>`
- `ShadowNode::UnsharedListOfWeak` → `std::shared_ptr<std::vector<std::weak_ptr<const ShadowNode>>>`

the type aliases will be deleted later.

Reviewed By: lenaic

Differential Revision: D90331677
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported meta-exported p: Facebook Partner: Facebook Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants