console: use spread notation instead of Object.assign

BridgeAR · targos · commit 561c2689efd7 · 2019-01-01T14:38:15.000+01:00
PR-URL: #25149 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Denys Otrishko <shishugi@gmail.com>
diff --git a/lib/internal/console/constructor.js b/lib/internal/console/constructor.js
@@ -411,11 +411,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));

-Original file line number
+Diff line change
 constfinal=(k,v)=>this.log(cliTable(k,v));
 constinspect=(v)=>{
 -constopt={depth: 0,maxArrayLength: 3};
 -if(v!==null&&typeofv==='object'&&
 -!isArray(v)&&ObjectKeys(v).length>2)
 -opt.depth=-1;
 -Object.assign(opt,this[kGetInspectOptions](this._stdout));
 +constdepth=v!==null&&
 +typeofv==='object'&&
 +!isArray(v)&&
 +ObjectKeys(v).length>2 ? -1 : 0;
 +constopt={
 + depth,
 +maxArrayLength: 3,
 + ...this[kGetInspectOptions](this._stdout)
 +};
 returnutil.inspect(v,opt);
 };
 constgetIndexArray=(length)=>ArrayFrom({ length },(_,i)=>inspect(i));