From b1e6f2c07142c9ac39358253a8589a46bf89cc45 Mon Sep 17 00:00:00 2001 From: Ruben Bridgewater Date: Wed, 12 Dec 2018 05:59:18 +0100 Subject: [PATCH] util: remove todo Most people are going to use the existing option and switching the name now comes with a cost which does not seem to justify the improvement. --- doc/api/util.md | 4 ---- lib/internal/util/inspect.js | 2 -- 2 files changed, 6 deletions(-) diff --git a/doc/api/util.md b/doc/api/util.md index 8c21bef9aed0be..69cda8e6b77864 100644 --- a/doc/api/util.md +++ b/doc/api/util.md @@ -446,10 +446,6 @@ changes: * `showProxy` {boolean} If `true`, then objects and functions that are `Proxy` objects will be introspected to show their `target` and `handler` objects. **Default:** `false`. - * `maxArrayLength` {integer} Specifies the maximum number of `Array`, [`TypedArray`][], [`WeakMap`][] and [`WeakSet`][] elements to include when formatting. Set to `null` or `Infinity` to show all elements. Set to `0` or diff --git a/lib/internal/util/inspect.js b/lib/internal/util/inspect.js index 6ec27b9d1d9c05..2c01d69a939d67 100644 --- a/lib/internal/util/inspect.js +++ b/lib/internal/util/inspect.js @@ -162,8 +162,6 @@ function inspect(value, opts) { colors: inspectDefaultOptions.colors, customInspect: inspectDefaultOptions.customInspect, showProxy: inspectDefaultOptions.showProxy, - // TODO(BridgeAR): Deprecate `maxArrayLength` and replace it with - // `maxEntries`. maxArrayLength: inspectDefaultOptions.maxArrayLength, breakLength: inspectDefaultOptions.breakLength, compact: inspectDefaultOptions.compact,