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

Set _interfaceState Before Calling fetchData et al. - #914

Merged
appleguy merged 1 commit into
facebookarchive:masterfrom
Adlai-Holler:UpdateInterfaceStateBeforeCallout
Dec 6, 2015
Merged

appleguy merged 1 commit into
facebookarchive:masterfrom
Adlai-Holler:UpdateInterfaceStateBeforeCallout

Conversation

@Adlai-Holler

Copy link
Copy Markdown
Contributor

No description provided.

@Adlai-Holler

Copy link
Copy Markdown
Contributor Author

Simulator fail.

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.

Great idea to minimize the critical section.

@appleguy

appleguy commented Dec 6, 2015

Copy link
Copy Markdown
Contributor

Nice improvement. Clearly it makes sense to have the new value available during subclass handling of the various methods called - as well as have the property lock free.

appleguy added a commit that referenced this pull request Dec 6, 2015
…allout

Minimize critical section in -setInterfaceState:, and move ivar set before subclass hooks are called.
@appleguy
appleguy merged commit faf9d26 into facebookarchive:master Dec 6, 2015
peter-iakovlev pushed a commit to peter-iakovlev/AsyncDisplayKit that referenced this pull request Jul 21, 2018
…rchive#924)

* fix SIMULATE_WEB_RESPONSE not imported facebookarchive#449

* Fix to make rangeMode update in right time

* remove uncessary assert
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.

3 participants