[Fix] use has-symbols package to ensure we support Symbol shams too.

ljharb · May 31, 2017 · 75135c8 · 75135c8
1 parent 84bccf3
commit 75135c8
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/implementation.js b/implementation.js
@@ -19,7 +19,7 @@ var slice = Function.call.bind(Array.prototype.slice);
 var concat = Function.call.bind(Array.prototype.concat);
 var forEach = Function.call.bind(Array.prototype.forEach);
 
-var hasSymbols = typeof Symbol === 'function' && typeof Symbol('') === 'symbol';
+var hasSymbols = require('has-symbols')();
 
 var kCustomPromisifiedSymbol = hasSymbols ? Symbol('util.promisify.custom') : null;
 var kCustomPromisifyArgsSymbol = hasSymbols ? Symbol('customPromisifyArgs') : null;

diff --git a/package.json b/package.json
@@ -5,6 +5,7 @@
 	"main": "index.js",
 	"dependencies": {
 		"define-properties": "^1.1.2",
+		"has-symbols": "^1.0.0",
 		"object.getownpropertydescriptors": "^2.0.3"
 	},
 	"devDependencies": {