ReactiveX / rxjs / 4203

97%

push

travis-ci

fix(Subscription): fold ChildSubscription logic into Subscriber to prevent operators from leaking ChildSubscriptions. (#2360)

The addition of ChildSubscription to fix #2244 accidentally introduced a different memory leak. Most operators that add and remove inner Subscriptions store the inner Subscriber instance, not the value returned by Subscription#add. When they try to remove the inner Subscription manually, nothing is removed, because the ChildSubscription wrapper instance is the one added to the subscriptions list.

Fixes #2355

84 of 84 new or added lines in 8 files covered. (100.0%)

6517 of 6672 relevant lines covered (97.68%)

672.33 hits per line