util: remove eslint comments and rename variables

This should improve the readability of the code. PR-URL: #25255 Reviewed-By: James M Snell <jasnell@gmail.com>
nodejs · Dec 28, 2018 · 46ec26f8aa6be7eb09cdcb15aad3afc0ce5c1a0f · 46ec26f
1 parent 7ff4410
commit 46ec26f8aa6be7eb09cdcb15aad3afc0ce5c1a0f
Unified Split

Showing with 5 additions and 6 deletions.

+5 −6 lib/internal/util/inspect.js
diff --git a/lib/internal/util/inspect.js b/lib/internal/util/inspect.js
@@ -822,12 +822,11 @@ function formatPrimitive(fn, value, ctx) {
    if (ctx.compact === false &&
      ctx.indentationLvl + value.length > ctx.breakLength &&
      value.length > kMinLineLength) {
-  
-      // eslint-disable-next-line max-len
-  
-      const minLineLength = Math.max(ctx.breakLength - ctx.indentationLvl, kMinLineLength);
-  
-      // eslint-disable-next-line max-len
-  
-      const averageLineLength = Math.ceil(value.length / Math.ceil(value.length / minLineLength));
+  
+      const rawMaxLineLength = ctx.breakLength - ctx.indentationLvl;
+  
+      const maxLineLength = Math.max(rawMaxLineLength, kMinLineLength);
+  
+      const lines = Math.ceil(value.length / maxLineLength);
+  
+      const averageLineLength = Math.ceil(value.length / lines);
      const divisor = Math.max(averageLineLength, kMinLineLength);
-  
-      let res = '';
      if (readableRegExps[divisor] === undefined) {
        // Build a new RegExp that naturally breaks text into multiple lines.
        //
@@ -843,7 +842,7 @@ function formatPrimitive(fn, value, ctx) {
      const matches = value.match(readableRegExps[divisor]);
      if (matches.length > 1) {
        const indent = ' '.repeat(ctx.indentationLvl);
-  
-        res += `${fn(strEscape(matches[0]), 'string')} +\n`;
+  
+        let res = `${fn(strEscape(matches[0]), 'string')} +\n`;
        for (var i = 1; i < matches.length - 1; i++) {
          res += `${indent}  ${fn(strEscape(matches[i]), 'string')} +\n`;
        }