From 6cc74b038fa0d2e64e8060415487706117b4a77c Mon Sep 17 00:00:00 2001 From: Ruben Bridgewater Date: Fri, 28 Dec 2018 02:42:27 +0100 Subject: [PATCH] util: remove eslint comments and rename variables This should improve the readability of the code. PR-URL: https://github.com/nodejs/node/pull/25255 Reviewed-By: James M Snell --- lib/internal/util/inspect.js | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/lib/internal/util/inspect.js b/lib/internal/util/inspect.js index e5cde4d3354a86..88362f8615f844 100644 --- 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`; }