util: use average bias while grouping arrays

BridgeAR · BridgeAR · commit 225fb1bff734 · 2019-06-17T21:12:34.000+02:00
This makes sure that strongly deviating entry length are taken into account while grouping arrays. PR-URL: #28070 Refs: #27690 Reviewed-By: James M Snell <jasnell@gmail.com>
diff --git a/lib/internal/util/inspect.js b/lib/internal/util/inspect.js
@@ -972,7 +972,8 @@ function groupArrayElements(ctx, output, value) {
       (totalLength / actualMax > 5 || maxLength <= 6)) {
 
     const approxCharHeights = 2.5;
-    const biasedMax = Math.max(actualMax - 4, 1);
+    const averageBias = Math.sqrt(actualMax - totalLength / output.length);
+    const biasedMax = Math.max(actualMax - 3 - averageBias, 1);
     // Dynamically check how many columns seem possible.
     const columns = Math.min(
       // Ideally a square should be drawn. We expect a character to be about 2.5
diff --git a/test/parallel/test-util-inspect.js b/test/parallel/test-util-inspect.js
@@ -2252,14 +2252,12 @@ assert.strictEqual(
 
   expected = [
     '[',
-    '  1, 1,         1,',
-    '  1, 1,         1,',
-    '  1, 1,         1,',
-    '  1, 1,         1,',
-    '  1, 1,         1,',
-    '  1, 1,         1,',
-    '  1, 1,         1,',
-    '  1, 1,         1,',
+    '  1, 1,         1, 1,',
+    '  1, 1,         1, 1,',
+    '  1, 1,         1, 1,',
+    '  1, 1,         1, 1,',
+    '  1, 1,         1, 1,',
+    '  1, 1,         1, 1,',
     '  1, 1, 123456789',
     ']'
   ].join('\n');
