util: remove eslint comments and rename variables

This should improve the readability of the code. PR-URL: #25255 Reviewed-By: James M Snell <[email protected]>
nodejs · Jan 14, 2019 · 46ec26f · 46ec26f
1 parent 7ff4410
commit 46ec26f
Showing 1 changed file with 5 additions and 6 deletions.
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`;
         }