-
Notifications
You must be signed in to change notification settings - Fork 270
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.
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
Improved CompositeSubscription #633
Conversation
Codecov Report
@@ Coverage Diff @@
## master #633 +/- ##
=======================================
Coverage 92.94% 92.94%
=======================================
Files 74 74
Lines 2225 2225
=======================================
Hits 2068 2068
Misses 157 157 |
/// Remove the subscription from this composite and cancel it if it has been removed. | ||
Future<void>? remove( | ||
StreamSubscription<dynamic> subscription, { | ||
bool canCancel = true, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we should use bool shouldCancel = true
:))
…w return a completion future - Fixed an issue where a stream not present in CompositeSubscription was canceled - Added the ability not to cancel the stream when it is removed from CompositeSubscription
3d0e7da
to
d2f33d4
Compare
@hoc081098 Wouldn't it be more correct to use |
I think |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your contribution!
dispose
,clear
, andremove
methods now return a completion future