doc,test: fix inspect's sorted compare function

In V8 7.0, the array sorting algorithm was changed to Timsort, which is stable. A compare function returning only `true` or `false` (converted to 0 and 1) cannot work properly. Backport-PR-URL: #23039 PR-URL: #22992 Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Kyle Farnung <[email protected]>
nodejs · Sep 24, 2018 · b48dc0b · b48dc0b
1 parent 68eaa87
commit b48dc0b
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/doc/api/util.md b/doc/api/util.md
@@ -554,7 +554,7 @@ const o1 = {
 };
 console.log(inspect(o1, { sorted: true }));
 // { a: '`a` comes before `b`', b: [ 2, 3, 1 ], c: Set { 1, 2, 3 } }
-console.log(inspect(o1, { sorted: (a, b) => a < b }));
+console.log(inspect(o1, { sorted: (a, b) => b.localeCompare(a) }));
 // { c: Set { 3, 2, 1 }, b: [ 2, 3, 1 ], a: '`a` comes before `b`' }
 
 const o2 = {

diff --git a/test/parallel/test-util-inspect.js b/test/parallel/test-util-inspect.js
@@ -1730,7 +1730,7 @@ assert.strictEqual(
 assert.strictEqual(
   inspect(
     { a200: 4, a100: 1, a102: 3, a101: 2 },
-    { sorted(a, b) { return a < b; } }
+    { sorted(a, b) { return b.localeCompare(a); } }
   ),
   '{ a200: 4, a102: 3, a101: 2, a100: 1 }'
 );