url: refactor lib/internal/url.js

* set an identifier for the separator rather than using multiple instances of the same literal * consistent arrow function body formatting PR-URL: #10912 Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Joyee Cheung <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Italo A. Casas <[email protected]> Reviewed-By: Timothy Gu <[email protected]>
nodejs · Jan 30, 2017 · c0c1a4c · c0c1a4c
1 parent 90a9917
commit c0c1a4c
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/lib/internal/url.js b/lib/internal/url.js
@@ -860,6 +860,7 @@ URLSearchParams.prototype[Symbol.iterator] = URLSearchParams.prototype.entries;
 
 URLSearchParams.prototype[util.inspect.custom] =
   function inspect(recurseTimes, ctx) {
+    const separator = ', ';
     const innerOpts = Object.assign({}, ctx);
     if (recurseTimes !== null) {
       innerOpts.depth = recurseTimes - 1;
@@ -872,13 +873,14 @@ URLSearchParams.prototype[util.inspect.custom] =
       output.push(`${innerInspect(list[i])} => ${innerInspect(list[i + 1])}`);
 
     const colorRe = /\u001b\[\d\d?m/g;
-    const length = output.reduce((prev, cur) => {
-      return prev + cur.replace(colorRe, '').length + ', '.length;
-    }, -', '.length);
+    const length = output.reduce(
+      (prev, cur) => prev + cur.replace(colorRe, '').length + separator.length,
+      -separator.length
+    );
     if (length > ctx.breakLength) {
       return `${this.constructor.name} {\n  ${output.join(',\n  ')} }`;
     } else if (output.length) {
-      return `${this.constructor.name} { ${output.join(', ')} }`;
+      return `${this.constructor.name} { ${output.join(separator)} }`;
     } else {
       return `${this.constructor.name} {}`;
     }