console: use spread notation instead of Object.assign

PR-URL: nodejs#25149 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Denys Otrishko <[email protected]>
peterwmwong · Dec 27, 2018 · 4ea2c1c · 4ea2c1c
1 parent 5ac30c9
commit 4ea2c1c
Showing 1 changed file with 9 additions and 5 deletions.
diff --git a/lib/internal/console/constructor.js b/lib/internal/console/constructor.js
@@ -434,11 +434,15 @@ Console.prototype.table = function(tabularData, properties) {
   const final = (k, v) => this.log(cliTable(k, v));
 
   const inspect = (v) => {
-    const opt = { depth: 0, maxArrayLength: 3 };
-    if (v !== null && typeof v === 'object' &&
-      !isArray(v) && ObjectKeys(v).length > 2)
-      opt.depth = -1;
-    Object.assign(opt, this[kGetInspectOptions](this._stdout));
+    const depth = v !== null &&
+                  typeof v === 'object' &&
+                  !isArray(v) &&
+                  ObjectKeys(v).length > 2 ? -1 : 0;
+    const opt = {
+      depth,
+      maxArrayLength: 3,
+      ...this[kGetInspectOptions](this._stdout)
+    };
     return util.inspect(v, opt);
   };
   const getIndexArray = (length) => ArrayFrom({ length }, (_, i) => inspect(i));