1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

util: remove todo

Browse Source 
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>
This commit is contained in:
Ruben Bridgewater 2018-12-12 05:59:18 +01:00 committed by Rich Trott
parent b32e5e08b2
commit 885de1ba70
2 changed files with 0 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
doc/api/util.md
View File

@ -449,10 +449,6 @@ changes:
* `showProxy` {boolean} If `true`, then objects and functions that are * `showProxy` {boolean} If `true`, then objects and functions that are
`Proxy` objects will be introspected to show their `target` and `handler` `Proxy` objects will be introspected to show their `target` and `handler`
objects. **Default:** `false`. objects. **Default:** `false`.
<!--
TODO(BridgeAR): Deprecate `maxArrayLength` and replace it with
`maxEntries`.
-->
* `maxArrayLength` {integer} Specifies the maximum number of `Array`, * `maxArrayLength` {integer} Specifies the maximum number of `Array`,
[`TypedArray`][], [`WeakMap`][] and [`WeakSet`][] elements to include when [`TypedArray`][], [`WeakMap`][] and [`WeakSet`][] elements to include when
formatting. Set to `null` or `Infinity` to show all elements. Set to `0` or formatting. Set to `null` or `Infinity` to show all elements. Set to `0` or

2
lib/internal/util/inspect.js
View File

@ -163,8 +163,6 @@ function inspect(value, opts) {
colors: inspectDefaultOptions.colors, colors: inspectDefaultOptions.colors,
customInspect: inspectDefaultOptions.customInspect, customInspect: inspectDefaultOptions.customInspect,
showProxy: inspectDefaultOptions.showProxy, showProxy: inspectDefaultOptions.showProxy,
// TODO(BridgeAR): Deprecate `maxArrayLength` and replace it with
// `maxEntries`.
maxArrayLength: inspectDefaultOptions.maxArrayLength, maxArrayLength: inspectDefaultOptions.maxArrayLength,
breakLength: inspectDefaultOptions.breakLength, breakLength: inspectDefaultOptions.breakLength,
compact: inspectDefaultOptions.compact, compact: inspectDefaultOptions.compact,