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

OperatorCombineLatest #1079

Merged

benjchristensen merged 1 commit into ReactiveX:master from akarnokd:OperatorCombineLatest

Apr 24, 2014

Member

akarnokd commented Apr 24, 2014

Operator CombineLatest

Issue #1060

Remarks:

Added an overload to Observable which accepts any number of sources. Not sure why it wasn't exposed before? Perhaps removed?
Relying on just the SO's onNext is not enough as leaving the synchronized block allows reordering of onNext calls. Therefore, the combinator uses the trick from SerializedObserver to make sure the combination order is consistent with whe winning of the synchronization "race".


          OperatorCombineLatest

817fb3a

akarnokd mentioned this pull request

Operations to rewrite #1060

Closed

57 tasks

cloudbees-pull-request-builder commented Apr 24, 2014

RxJava-pull-requests #990 SUCCESS
This pull request looks good

benjchristensen merged commit 817fb3a into ReactiveX:master

akarnokd deleted the OperatorCombineLatest branch

April 24, 2014 21:23

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment