Skip to content
This repository was archived by the owner on Feb 2, 2023. It is now read-only.

[ASLayoutSpec] Remove legacy and unused parent property#2987

Merged
Adlai-Holler merged 1 commit intofacebookarchive:masterfrom
maicki:MSRemoveLegacyParentPointer
Feb 7, 2017
Merged

[ASLayoutSpec] Remove legacy and unused parent property#2987
Adlai-Holler merged 1 commit intofacebookarchive:masterfrom
maicki:MSRemoveLegacyParentPointer

Conversation

@maicki
Copy link
Copy Markdown
Contributor

@maicki maicki commented Feb 6, 2017

No description provided.


/**
* Parent of the layout spec
*/
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does this need to be deprecated?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It was never used outside of the framework (and actually in the framework it was not used either if you want to be precise). You can argue it was publicly available, but there was no reason to use it in the first place.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, it's pretty unlikely anyone was using this.

@Adlai-Holler Adlai-Holler merged commit aedac5c into facebookarchive:master Feb 7, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants