Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix crash when diffing to/from empty/nil states to/from non-empty/nil states #125

Merged
merged 1 commit into from
Jul 16, 2018

Conversation

jessesquires
Copy link
Contributor

Closes #124
Internal: IP-3275

@jessesquires jessesquires added this to the 0.1.2 milestone Jul 9, 2018
@ghost
Copy link

ghost commented Jul 9, 2018

@jessesquires jessesquires requested a review from anayini July 9, 2018 23:30
@ghost ghost assigned anayini Jul 9, 2018
anayini
anayini previously approved these changes Jul 9, 2018
Copy link
Contributor

@anayini anayini left a comment

Choose a reason for hiding this comment

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

💯

@jessesquires
Copy link
Contributor Author

Ok. Pushed more fixes. (Tests were failing)

I can consistently crash UIKit because Dwifft is not producing correct diffs.

That's out of scope for this PR right now though. We'll have to live with it until we can switch to IGListKit.

@asmallteapot
Copy link

@jessesquires Is the crashing consistent enough that we could throw an exception from ReactiveLists to make its source clearer?

@jessesquires jessesquires force-pushed the jsq/fix-empty-state-crash branch from d552263 to b7f8be1 Compare July 13, 2018 22:40
@jessesquires
Copy link
Contributor Author

Follow-up at #126

1 similar comment
@jessesquires
Copy link
Contributor Author

Follow-up at #126

@jessesquires jessesquires merged commit a431d76 into master Jul 16, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants