Combine should support observables
Closed this issue · 3 comments
hahn-kev commented
I was a bit surprised to find that this doesn't work:
Observable<void> o;
auto all = combine() << o;
sure you can just call o.future()
but it seems like that should be overloaded in combine.
Could this just be added to the Combinator
?
template <typename T>
Combinator& operator<<(Observable<T> observable)
{
combinedFuture->addFuture(observable.future());
return *this;
}
benlau commented
Added in master branch. Will release later
benlau commented
It is already released with 0.3.6.1