Skip to content

toFuture incorrect diagram/description #6198

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
Closed
akarnokd opened this issue Sep 3, 2018 · 0 comments · Fixed by #6200
Closed

toFuture incorrect diagram/description #6198

akarnokd opened this issue Sep 3, 2018 · 0 comments · Fixed by #6200
Labels
2.x Documentation PR welcome
Milestone
2.2 backlog

Comments

@akarnokd
Copy link
Member

akarnokd commented Sep 3, 2018

Both Flowable.toFuture and Observable.toFuture describes the wrong behavior: If the source has more than one element, the outcome is actually IndexOutOfBoundsException.

In addition, the docs should mention the source has to terminate in order for the Future to terminate.

Plus, the marble should be updated to reflect the actual behavior (there is no dispose upon the first received item).

image
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
2.x Documentation PR welcome
Projects
None yet
Milestone
2.2 backlog
Development

Successfully merging a pull request may close this issue.

2.x: Fix toFuture marbles and descriptions akarnokd/RxJava
1 participant
@akarnokd