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.

Sign up for GitHub

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

Jump to bottom

2.x: coverage, fixes, cleanup 10/20-2 #4738

Merged
merged 2 commits into from Oct 20, 2016
Merged

2.x: coverage, fixes, cleanup 10/20-2 #4738

merged 2 commits into from Oct 20, 2016

Conversation

akarnokd
Copy link
Member

I cut this short so there is time for the 2.0.0-RC release preparations.

  • Introduce SimplePlainQueue where poll doesn't throw, avoiding the need for try-catches where the queue is one of the standard lock-free queues.
  • Fix FlatMap error, cancellation and resource management.
  • Coverage of some Flowable operators
  • Removal of impossible and unused code paths.

@akarnokd akarnokd added this to the 2.0 RC5 milestone Oct 20, 2016
@akarnokd akarnokd mentioned this pull request Oct 20, 2016
Closed
@akarnokd akarnokd merged commit d2c9c34 into ReactiveX:2.x Oct 20, 2016
@akarnokd akarnokd deleted the Coverage_10_20_2 branch October 20, 2016 20:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JakeWharton JakeWharton JakeWharton approved these changes

Assignees
No one assigned
Labels
Bug Cleanup Coverage
Projects
None yet
Milestone
2.0 RC5
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@akarnokd @JakeWharton