Use static Symbol check

skeggse · Oct 18, 2018 · f231550 · f231550
1 parent 774ed58
commit f231550
Showing 1 changed file with 6 additions and 23 deletions.
diff --git a/lib/index.js b/lib/index.js
@@ -197,31 +197,14 @@ internals.checkIpV6 = function (items) {
 };
 
 
-internals.isIterable = function (value) {
+internals.isIterable = Array.isArray;
 
-    /* $lab:coverage:off$ */
-    if (!value) {
-        return false;
-    }
-    /* $lab:coverage:on$ */
-
-    if (typeof value !== 'object') {
-        return false;
-    }
-
-    // Return early in case we're on a version that doesn't support iterators.
-    if (Array.isArray(value)) {
-        return true;
-    }
-
-    /* $lab:coverage:off$ */
-    if (typeof Symbol === 'undefined') {
-        return false;
-    }
-    /* $lab:coverage:on$ */
 
-    return typeof value[Symbol.iterator] === 'function';
-};
+/* $lab:coverage:off$ */
+if (typeof Symbol !== 'undefined') {
+    internals.isIterable = (value) => Array.isArray(value) || (!!value && typeof value === 'object' && typeof value[Symbol.iterator] === 'function');
+}
+/* $lab:coverage:on$ */
 
 
 // Node 10 introduced isSet and isMap, which are useful for cross-context type