Issue #1642 Fix null-emitting combineLatest

[edenman]

@benjchristensen

[dpsm]

LGTM

[benjchristensen]

Thanks for fixing this. Stupid bug on my part.

It should have been:

buffer.getValue(o);
// or
buffer.accept(o, child);

Your fix does the same thing.

[edenman]

@benjchristensen just saw that this didn't make it into 0.20.5. Do I need to do something to make sure it gets into 0.20.6? Can't tell how you guys are managing the various release branches.

[benjchristensen]

This was merged into 1.x and is in the 1.0 releases. (it was in 1.0.0-rc.4 https://github.com/ReactiveX/RxJava/releases/tag/v1.0.0-rc.4)

If you want it also in the 0.20.x releases go ahead and submit another PR against that branch.

1.x is where most of the focus is as we work towards the 1.0 Final release scheduled for November 18th.

[edenman]

Ah. I tried switching us over to 1.0.0-rc4 but one of our dependencies uses the com.netflix packaging and Android won't let us include both in our apk (dex complains about duplicate classes). #1761

[benjchristensen]

one of our dependencies uses the com.netflix packaging

Yes, it's going to be a painful one-time migration to get onto 1.0. Once we hit 1.0 Final I'll encourage everyone to do so.