Skip to content
This repository has been archived by the owner on Mar 16, 2021. It is now read-only.

User addAll instead of using an fori #63

Closed
StefMa opened this issue Jan 24, 2017 · 1 comment · Fixed by #64
Closed

User addAll instead of using an fori #63

StefMa opened this issue Jan 24, 2017 · 1 comment · Fixed by #64
Labels
rx

Comments

@StefMa
Copy link
Contributor

StefMa commented Jan 24, 2017

Currently we are using an fori loop to add subscriptions to the CompositeSubscprion handler.
See here.

I found out that the CompositeSubscrption (and the RxJava2 part) have a addAll(...) method.
Think about to use these one instead of our own fori loop
@StefMa StefMa added the rx label Jan 24, 2017
@StefMa StefMa mentioned this issue Jan 24, 2017
Merged
@StefMa
Copy link
Contributor Author

StefMa commented Jan 24, 2017

Oh wow. Cool 😎. This issue was automatically closed when I've merged #64

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
rx
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Use addAll instead of our own fori
1 participant
@StefMa