fix(Observable.toArray): Fix Observable.toArray with multiple subscriptions.

[josketres]

Description:
The current implementation of Observable.toArray reuses the same array instance for multiple subscriptions.

This example clarifies why that's wrong.

const Rx = require('rxjs/Rx');
const observable = Rx.Observable.from([1,2]).toArray();
observable.take(1).subscribe(console.log); // prints [ 1, 2 ]
observable.take(1).subscribe(console.log); // prints [ 1, 2, 1, 2 ]

[rxjs-bot]

	Messages
📖

CJS: 1381.3KB, global: 750.6KB (gzipped: 120.4KB), min: 145.3KB (gzipped: 31.4KB)

Generated by 🚫 dangerJS

[coveralls]

Coverage increased (+0.0008%) to 97.079% when pulling b5ce3ff on josketres:fix-to-array-operator-multiple-subscriptions into 515a753 on ReactiveX:master.

[coveralls]

Coverage increased (+0.0008%) to 97.079% when pulling b5ce3ff on josketres:fix-to-array-operator-multiple-subscriptions into 515a753 on ReactiveX:master.

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.