diff options
author | Ruben Bridgewater <ruben@bridgewater.de> | 2018-12-12 05:59:18 +0100 |
---|---|---|
committer | Rich Trott <rtrott@gmail.com> | 2018-12-14 21:32:51 -0800 |
commit | 885de1ba70094905ccce4825e7875fc0ec5740b2 (patch) | |
tree | 6f48fec00b5cee3c67f47da1d4373aedf0e622de | |
parent | b32e5e08b2c6dc1390d4bab1fe4e2d3670d251c8 (diff) | |
download | android-node-v8-885de1ba70094905ccce4825e7875fc0ec5740b2.tar.gz android-node-v8-885de1ba70094905ccce4825e7875fc0ec5740b2.tar.bz2 android-node-v8-885de1ba70094905ccce4825e7875fc0ec5740b2.zip |
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.
PR-URL: https://github.com/nodejs/node/pull/24982
Reviewed-By: Michaƫl Zasso <targos@protonmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Anto Aravinth <anto.aravinth.cse@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
-rw-r--r-- | doc/api/util.md | 4 | ||||
-rw-r--r-- | lib/internal/util/inspect.js | 2 |
2 files changed, 0 insertions, 6 deletions
diff --git a/doc/api/util.md b/doc/api/util.md index 0b9e477394..22859dfdfe 100644 --- a/doc/api/util.md +++ b/doc/api/util.md @@ -449,10 +449,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`. - <!-- - TODO(BridgeAR): Deprecate `maxArrayLength` and replace it with - `maxEntries`. - --> * `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 f7e5507882..4e8e4d4730 100644 --- a/lib/internal/util/inspect.js +++ b/lib/internal/util/inspect.js @@ -163,8 +163,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, |