lib: combine contructor, tag, Object into a function

combine these parts into a function to be used in multiple parts PR-URL: #24171 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]>
nodejs · Nov 13, 2018 · 6f80a45 · 6f80a45
1 parent 825f0dd
commit 6f80a45
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/lib/internal/util/inspect.js b/lib/internal/util/inspect.js
@@ -400,6 +400,10 @@ function getKeys(value, showHidden) {
   return keys;
 }
 
+function getCtxStyle(constructor, tag) {
+  return constructor || tag || 'Object';
+}
+
 function formatProxy(ctx, proxy, recurseTimes) {
   if (recurseTimes != null) {
     if (recurseTimes < 0)
@@ -741,7 +745,7 @@ function formatRaw(ctx, value, recurseTimes) {
 
   if (recurseTimes != null) {
     if (recurseTimes < 0)
-      return ctx.stylize(`[${constructor || tag || 'Object'}]`, 'special');
+      return ctx.stylize(`[${getCtxStyle(constructor, tag)}]`, 'special');
     recurseTimes -= 1;
   }
 
@@ -793,7 +797,7 @@ function handleMaxCallStackSize(ctx, err, constructor, tag, indentationLvl) {
     ctx.seen.pop();
     ctx.indentationLvl = indentationLvl;
     return ctx.stylize(
-      `[${constructor || tag || 'Object'}: Inspection interrupted ` +
+      `[${getCtxStyle(constructor, tag)}: Inspection interrupted ` +
         'prematurely. Maximum call stack size exceeded.]',
       'special'
     );