Pull 267 - Merge combineLatest

[benjchristensen]

Spent some time reviewing it and it seems good. Any bugs can be dealt with as people start playing with it instead of me holding this up as a pull request any longer since this exposes combineLatest in Observable for I believe the first time and is thus a low risk to breaking any existing users.

[cloudbees-pull-request-builder]

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

[mttkay]

I noticed there is no marble diagram in the documentation, I find these extremely helpful to understand how an operator works. Is combineLatest different from zip only in how events are timed? It looks like it accomplishes something very similar.

[jmhofer]

It's different in that it doesn't keep the sources "synced". How do you create those diagrams? I'd draw one if I knew how...

[DavidMGross]

There's a marble diagram for combineLatest at:
https://github.com/Netflix/RxJava/wiki/Combining-Observables#combinelatest
and it should also be in the javadocs as of the next release.

On Sat, Jul 20, 2013 at 1:37 AM, Joachim Hofer notifications@github.amrom.workers.devwrote:

It's different in that it doesn't keep the sources "synced". How do you
create those diagrams? I'd draw one if I knew how...

—
Reply to this email directly or view it on GitHubhttps://github.com//pull/303#issuecomment-21290348
.

[mttkay]

Thanks, that clarifies it a lot!