diff options
author | Ruben Bridgewater <ruben@bridgewater.de> | 2019-11-30 11:04:34 +0100 |
---|---|---|
committer | Ruben Bridgewater <ruben@bridgewater.de> | 2019-12-07 00:38:28 +0100 |
commit | c430aebe8683699fc072cb87e5be03af370a3f2f (patch) | |
tree | 3f93ee11527492d5295853b93c0252a4afafa423 /lib/internal/util | |
parent | bdb1083ef54cf99c959c554b741e62943b56e2af (diff) | |
download | android-node-v8-c430aebe8683699fc072cb87e5be03af370a3f2f.tar.gz android-node-v8-c430aebe8683699fc072cb87e5be03af370a3f2f.tar.bz2 android-node-v8-c430aebe8683699fc072cb87e5be03af370a3f2f.zip |
util: improve performance inspecting proxies
This makes sure we do not retrieve the handler in case it's not
required. This improves the performance a tiny bit for these cases.
PR-URL: https://github.com/nodejs/node/pull/30767
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Diffstat (limited to 'lib/internal/util')
-rw-r--r-- | lib/internal/util/inspect.js | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/internal/util/inspect.js b/lib/internal/util/inspect.js index 7005911621..4e6f967c6a 100644 --- a/lib/internal/util/inspect.js +++ b/lib/internal/util/inspect.js @@ -643,12 +643,12 @@ function formatValue(ctx, value, recurseTimes, typedArray) { const context = value; // Always check for proxies to prevent side effects and to prevent triggering // any proxy handlers. - const proxy = getProxyDetails(value); + const proxy = getProxyDetails(value, !!ctx.showProxy); if (proxy !== undefined) { if (ctx.showProxy) { return formatProxy(ctx, proxy, recurseTimes); } - value = proxy[0]; + value = proxy; } // Provide a hook for user-specified inspect functions. |