util: add fast internal array join method

BridgeAR · BridgeAR · commit 01652ccc682d · 2017-09-14T21:46:33.000-03:00
PR-URL: nodejs#14881 Reviewed-By: Refael Ackermann <refack@gmail.com> Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
diff --git a/lib/internal/util.js b/lib/internal/util.js
@@ -257,6 +257,20 @@ function promisify(original) {
 
 promisify.custom = kCustomPromisifiedSymbol;
 
+// The build-in Array#join is slower in v8 6.0
+function join(output, separator) {
+  var str = '';
+  if (output.length !== 0) {
+    for (var i = 0; i < output.length - 1; i++) {
+      // It is faster not to use a template string here
+      str += output[i];
+      str += separator;
+    }
+    str += output[i];
+  }
+  return str;
+}
+
 module.exports = {
   assertCrypto,
   cachedResult,
@@ -270,6 +284,7 @@ module.exports = {
   normalizeEncoding,
   objectToString,
   promisify,
+  join,
 
   // Symbol used to customize promisify conversion
   customPromisifyArgs: kCustomPromisifyArgsSymbol,
