use mock weakset until nodejs/node#6180 is fixed

bernardmcmanus · bernardmcmanus · commit a0b9da2a7bbc · 2017-05-28T02:30:26.000-04:00
diff --git a/lib/weak-set.js b/lib/weak-set.js
@@ -1,5 +1,3 @@
-const { WeakSet } = global;
-
 /**
  * IMPORTANT: MockWeakSet intentionally omits unused methods (like delete)
  */
@@ -15,5 +13,7 @@ function MockWeakSet(mws = []) {
 }
 
 export default function MaybeWeakSet(iterable) {
-	return WeakSet ? new WeakSet(iterable) : MockWeakSet(iterable);
+	// WeakSet causes a horrible memory leak in node so just use the mock for now
+	// https://github.com/nodejs/node/issues/6180
+	return MockWeakSet(iterable);
 }
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "bitclock",
-  "version": "0.2.0-beta.1",
+  "version": "0.2.0-beta.2",
   "license": "Apache-2.0",
   "repository": {
     "type": "git",

Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,3 @@`
`1`		`-const { WeakSet } = global;`
`2`		`-`
`3`	`1`	`/**`
`4`	`2`	`* IMPORTANT: MockWeakSet intentionally omits unused methods (like delete)`
`5`	`3`	`*/`
`@@ -15,5 +13,7 @@ function MockWeakSet(mws = []) {`
`15`	`13`	`}`
`16`	`14`
`17`	`15`	`export default function MaybeWeakSet(iterable) {`
`18`		`- return WeakSet ? new WeakSet(iterable) : MockWeakSet(iterable);`
	`16`	`+ // WeakSet causes a horrible memory leak in node so just use the mock for now`
	`17`	`+ // https://github.com/nodejs/node/issues/6180`
	`18`	`+ return MockWeakSet(iterable);`
`19`	`19`	`}`