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

Task improvements #202

Merged
merged 1 commit into from
Aug 8, 2016
Merged

Task improvements #202

merged 1 commit into from
Aug 8, 2016

Conversation

alexandru
Copy link
Member

Some task improvements:

  • For Callback, removed the popBeforeCall and async builders, because they were confusing
  • Instead Callback now has asyncOnSuccess and asyncOnError specified as extension methods (not methods on the class)
  • Task.create was changed to be safe, in that users are protected from stack-overflows on usage of onSuccess - this isn't always a good solution for performance reasons, but users knowing what they are doing always have unsafeCreate
  • Optimized chooseFirstOfList and ensured its stack safe - as part of it the implementation of CompositeCancelable was replaced with one that uses an internal mutable HashSet
  • Optimized TestScheduler and as a result some tests are running dramatically faster

@alexandru alexandru added this to the 2.0 milestone Aug 8, 2016
@alexandru alexandru self-assigned this Aug 8, 2016
@alexandru alexandru merged commit 12597dd into master Aug 8, 2016
@alexandru alexandru deleted the task-improvements branch November 8, 2016 17:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant