refactor(distinct): Remove Set polyfill

BREAKING CHANGE: Using `distinct` requires a `Set` implementation and must be polyfilled in older runtimes
ReactiveX · Nov 3, 2017 · 68ee499 · 68ee499
1 parent 5eb6af7
commit 68ee499
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 111 deletions.
diff --git a/spec/util/Set-spec.ts b/spec/util/Set-spec.ts
diff --git a/spec/util/root-spec.ts b/spec/util/root-spec.ts
diff --git a/src/operators/distinct.ts b/src/operators/distinct.ts
@@ -5,9 +5,12 @@ import { TeardownLogic } from '../Subscription';
 import { OuterSubscriber } from '../OuterSubscriber';
 import { InnerSubscriber } from '../InnerSubscriber';
 import { subscribeToResult } from '../util/subscribeToResult';
-import { ISet, Set } from '../util/Set';
 import { MonoTypeOperatorFunction } from '../interfaces';
 
+if (!Set) {
+  throw new Error('Set is not present, please polyfill');
+}
+
 /**
  * Returns an Observable that emits all items emitted by the source Observable that are distinct by comparison from previous items.
  *
@@ -73,7 +76,7 @@ class DistinctOperator<T, K> implements Operator<T, T> {
  * @extends {Ignored}
  */
 export class DistinctSubscriber<T, K> extends OuterSubscriber<T, T> {
-  private values: ISet<K> = new Set<K>();
+  private values = new Set<K>();
 
   constructor(destination: Subscriber<T>, private keySelector: (value: T) => K, flushes: Observable<any>) {
     super(destination);

diff --git a/src/util/Set.ts b/src/util/Set.ts