util: change %o depth default

Since the default for depth is changed to `20` it is logical to change the %o default as well. It will now always use the default depth. PR-URL: #17907 Refs: #12693 PR-URL: #22846 Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Denys Otrishko <[email protected]> Reviewed-By: Roman Reiss <[email protected]>
nodejs · Oct 2, 2018 · 1a1fe53 · 1a1fe53
1 parent ac7450a
commit 1a1fe53
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/doc/api/util.md b/doc/api/util.md
@@ -183,6 +183,10 @@ property take precedence over `--trace-deprecation` and
 <!-- YAML
 added: v0.5.3
 changes:
+  - version: REPLACEME
+    pr-url: https://github.com/nodejs/node/pull/17907
+    description: The `%o` specifier's `depth` option will now fall back to the
+                 default depth.
   - version: v8.4.0
     pr-url: https://github.com/nodejs/node/pull/14558
     description: The `%o` and `%O` specifiers are supported now.

diff --git a/lib/util.js b/lib/util.js
@@ -113,8 +113,7 @@ function formatWithOptions(inspectOptions, f) {
           {
             const opts = Object.assign({}, inspectOptions, {
               showHidden: true,
-              showProxy: true,
-              depth: 4
+              showProxy: true
             });
             tempStr = inspect(arguments[a++], opts);
             break;