errors,console: refactor to use ES2021 syntax

PR-URL: #42872 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Darshan Sen <[email protected]>

Author: xtx1130

lib/internal/console/constructor.js

@@ -100,7 +100,7 @@ function Console(options /* or: stdout, stderr, ignoreErrors = true */){
 // We have to test new.target here to see if this function is called
 // with new, because we need to define a custom instanceof to accommodate
 // the global console.
-if(!new.target){
+if(new.target===undefined){
 returnReflectConstruct(Console,arguments);
 }
 
@@ -486,7 +486,7 @@ const consoleMethods ={
 if(tabularData===null||typeoftabularData!=='object')
 returnthis.log(tabularData);
 
-if(cliTable===undefined)cliTable=require('internal/cli_table');
+cliTable??=require('internal/cli_table');
 constfinal=(k,v)=>this.log(cliTable(k,v));
 
 const_inspect=(v)=>{
@@ -570,8 +570,7 @@ const consoleMethods ={
 }else{
 constkeys=properties||ObjectKeys(item);
 for(constkeyofkeys){
-if(map[key]===undefined)
-map[key]=[];
+map[key]??=[];
 if((primitive&&properties)||
 !ObjectPrototypeHasOwnProperty(item,key))
 map[key][i]='';

lib/internal/errors.js

@@ -175,24 +175,19 @@ let assert;
 
 letinternalUtil=null;
 functionlazyInternalUtil(){
-if(!internalUtil){
-internalUtil=require('internal/util');
-}
+internalUtil??=require('internal/util');
 returninternalUtil;
 }
 
 letinternalUtilInspect=null;
 functionlazyInternalUtilInspect(){
-if(!internalUtilInspect){
-internalUtilInspect=require('internal/util/inspect');
-}
+internalUtilInspect??=require('internal/util/inspect');
 returninternalUtilInspect;
 }
 
 letbuffer;
 functionlazyBuffer(){
-if(buffer===undefined)
-buffer=require('buffer').Buffer;
+buffer??=require('buffer').Buffer;
 returnbuffer;
 }
 
@@ -411,7 +406,7 @@ function E(sym, val, def, ...otherClasses){
 functiongetMessage(key,args,self){
 constmsg=messages.get(key);
 
-if(assert===undefined)assert=require('internal/assert');
+assert??=require('internal/assert');
 
 if(typeofmsg==='function'){
 assert(
@@ -439,19 +434,15 @@ function getMessage(key, args, self){
 letuvBinding;
 
 functionlazyUv(){
-if(!uvBinding){
-uvBinding=internalBinding('uv');
-}
+uvBinding??=internalBinding('uv');
 returnuvBinding;
 }
 
 constuvUnmappedError=['UNKNOWN','unknown error'];
 
 functionuvErrmapGet(name){
 uvBinding=lazyUv();
-if(!uvBinding.errmap){
-uvBinding.errmap=uvBinding.getErrorMap();
-}
+uvBinding.errmap??=uvBinding.getErrorMap();
 returnMapPrototypeGet(uvBinding.errmap,name);
 }
 
@@ -578,7 +569,7 @@ const errnoException = hideStackFrames(
 // getSystemErrorName(err) to guard against invalid arguments from users.
 // This can be replaced with [ code ] = errmap.get(err) when this method
 // is no longer exposed to user land.
-if(util===undefined)util=require('util');
+util??=require('util');
 constcode=util.getSystemErrorName(err);
 constmessage=original ?
 `${syscall}${code}${original}` : `${syscall}${code}`;
@@ -612,7 +603,7 @@ const exceptionWithHostPort = hideStackFrames(
 // getSystemErrorName(err) to guard against invalid arguments from users.
 // This can be replaced with [ code ] = errmap.get(err) when this method
 // is no longer exposed to user land.
-if(util===undefined)util=require('util');
+util??=require('util');
 constcode=util.getSystemErrorName(err);
 letdetails='';
 if(port&&port>0){
@@ -1224,7 +1215,7 @@ E('ERR_INVALID_ARG_TYPE',
 }elseif(typeofactual==='function'&&actual.name){
 msg+=`. Received function ${actual.name}`;
 }elseif(typeofactual==='object'){
-if(actual.constructor&&actual.constructor.name){
+if(actual.constructor?.name){
 msg+=`. Received an instance of ${actual.constructor.name}`;
 }else{
 constinspected=lazyInternalUtilInspect()
@@ -1320,7 +1311,7 @@ E('ERR_INVALID_RETURN_PROPERTY_VALUE', (input, name, prop, value) =>{
 },TypeError);
 E('ERR_INVALID_RETURN_VALUE',(input,name,value)=>{
 lettype;
-if(value&&value.constructor&&value.constructor.name){
+if(value?.constructor?.name){
 type=`instance of ${value.constructor.name}`;
 }else{
 type=`type ${typeofvalue}`;