errors: move functions to error code

This makes sure the functions are actually directly beneath the
specification of an error code.
That way it is not necessary to jump around when looking at the
functionality.
PR-URL: #20486
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>

Author: BridgeAR

lib/internal/errors.js

@@ -410,6 +410,26 @@ function isStackOverflowError(err) {
 err.message===maxStack_ErrorMessage;
 }
 
+functiononeOf(expected,thing){
+assert(typeofthing==='string','`thing` has to be of type string');
+if(Array.isArray(expected)){
+constlen=expected.length;
+assert(len>0,
+'At least one expected value needs to be specified');
+expected=expected.map((i)=>String(i));
+if(len>2){
+return`one of ${thing}${expected.slice(0,len-1).join(', ')}, or `+
+expected[len-1];
+}elseif(len===2){
+return`one of ${thing}${expected[0]} or ${expected[1]}`;
+}else{
+return`of ${thing}${expected[0]}`;
+}
+}else{
+return`of ${thing}${String(expected)}`;
+}
+}
+
 module.exports={
  dnsException,
  errnoException,
@@ -444,7 +464,15 @@ E('ERR_ARG_NOT_ITERABLE', '%s must be iterable', TypeError);
 E('ERR_ASSERTION','%s',Error);
 E('ERR_ASYNC_CALLBACK','%s must be a function',TypeError);
 E('ERR_ASYNC_TYPE','Invalid name for async "type": %s',TypeError);
-E('ERR_BUFFER_OUT_OF_BOUNDS',bufferOutOfBounds,RangeError);
+E('ERR_BUFFER_OUT_OF_BOUNDS',
+// Using a default argument here is important so the argument is not counted
+// towards `Function#length`.
+(name=undefined)=>{
+if(name){
+return`"${name}" is outside of buffer bounds`;
+}
+return'Attempt to write outside buffer bounds';
+},RangeError);
 E('ERR_BUFFER_TOO_LARGE',
 `Cannot create a Buffer larger than 0x${kMaxLength.toString(16)} bytes`,
 RangeError);
@@ -582,7 +610,32 @@ E('ERR_INSPECTOR_CLOSED', 'Session was closed', Error);
 E('ERR_INSPECTOR_NOT_AVAILABLE','Inspector is not available',Error);
 E('ERR_INSPECTOR_NOT_CONNECTED','Session is not connected',Error);
 E('ERR_INVALID_ADDRESS_FAMILY','Invalid address family: %s',RangeError);
-E('ERR_INVALID_ARG_TYPE',invalidArgType,TypeError);
+E('ERR_INVALID_ARG_TYPE',
+(name,expected,actual)=>{
+assert(typeofname==='string',"'name' must be a string");
+
+// determiner: 'must be' or 'must not be'
+letdeterminer;
+if(typeofexpected==='string'&&expected.startsWith('not ')){
+determiner='must not be';
+expected=expected.replace(/^not/,'');
+}else{
+determiner='must be';
+}
+
+letmsg;
+if(name.endsWith(' argument')){
+// For cases like 'first argument'
+msg=`The ${name}${determiner}${oneOf(expected,'type')}`;
+}else{
+consttype=name.includes('.') ? 'property' : 'argument';
+msg=`The "${name}" ${type}${determiner}${oneOf(expected,'type')}`;
+}
+
+// TODO(BridgeAR): Improve the output by showing `null` and similar.
+msg+=`. Received type ${typeofactual}`;
+returnmsg;
+},TypeError);
 E('ERR_INVALID_ARG_VALUE',(name,value,reason='is invalid')=>{
 letinspected=util.inspect(value);
 if(inspected.length>128){
@@ -598,7 +651,16 @@ E('ERR_INVALID_ASYNC_ID', 'Invalid %s value: %s', RangeError);
 E('ERR_INVALID_BUFFER_SIZE',
 'Buffer size must be a multiple of %s',RangeError);
 E('ERR_INVALID_CALLBACK','Callback must be a function',TypeError);
-E('ERR_INVALID_CHAR',invalidChar,TypeError);
+E('ERR_INVALID_CHAR',
+// Using a default argument here is important so the argument is not counted
+// towards `Function#length`.
+(name,field=undefined)=>{
+letmsg=`Invalid character in ${name}`;
+if(field!==undefined){
+msg+=` ["${field}"]`;
+}
+returnmsg;
+},TypeError);
 E('ERR_INVALID_CURSOR_POS',
 'Cannot set cursor row without setting its column',TypeError);
 E('ERR_INVALID_DOMAIN_NAME','Unable to determine the domain name',TypeError);
@@ -648,7 +710,26 @@ E('ERR_IPC_DISCONNECTED', 'IPC channel is already disconnected', Error);
 E('ERR_IPC_ONE_PIPE','Child process can have only one IPC pipe',Error);
 E('ERR_IPC_SYNC_FORK','IPC cannot be used with synchronous forks',Error);
 E('ERR_METHOD_NOT_IMPLEMENTED','The %s method is not implemented',Error);
-E('ERR_MISSING_ARGS',missingArgs,TypeError);
+E('ERR_MISSING_ARGS',
+(...args)=>{
+assert(args.length>0,'At least one arg needs to be specified');
+letmsg='The ';
+constlen=args.length;
+args=args.map((a)=>`"${a}"`);
+switch(len){
+case1:
+msg+=`${args[0]} argument`;
+break;
+case2:
+msg+=`${args[0]} and ${args[1]} arguments`;
+break;
+default:
+msg+=args.slice(0,len-1).join(', ');
+msg+=`, and ${args[len-1]} arguments`;
+break;
+}
+return`${msg} must be specified`;
+},TypeError);
 E('ERR_MISSING_MODULE','Cannot find module %s',Error);
 E('ERR_MODULE_RESOLUTION_LEGACY',
 '%s not found by import in %s.'+
@@ -669,7 +750,13 @@ E('ERR_NO_CRYPTO',
 E('ERR_NO_ICU',
 '%s is not supported on Node.js compiled without ICU',TypeError);
 E('ERR_NO_LONGER_SUPPORTED','%s is no longer supported',Error);
-E('ERR_OUT_OF_RANGE',outOfRange,RangeError);
+E('ERR_OUT_OF_RANGE',
+(name,range,value)=>{
+letmsg=`The value of "${name}" is out of range.`;
+if(range!==undefined)msg+=` It must be ${range}.`;
+msg+=` Received ${value}`;
+returnmsg;
+},RangeError);
 E('ERR_REQUIRE_ESM','Must use import to load ES Module: %s',Error);
 E('ERR_SCRIPT_EXECUTION_INTERRUPTED',
 'Script execution was interrupted by `SIGINT`.',Error);
@@ -765,95 +852,3 @@ E('ERR_VM_MODULE_NOT_MODULE',
 'Provided module is not an instance of Module',Error);
 E('ERR_VM_MODULE_STATUS','Module status %s',Error);
 E('ERR_ZLIB_INITIALIZATION_FAILED','Initialization failed',Error);
-
-functioninvalidArgType(name,expected,actual){
-assert(typeofname==='string',"'name' must be a string");
-
-// determiner: 'must be' or 'must not be'
-letdeterminer;
-if(typeofexpected==='string'&&expected.startsWith('not ')){
-determiner='must not be';
-expected=expected.replace(/^not/,'');
-}else{
-determiner='must be';
-}
-
-letmsg;
-if(name.endsWith(' argument')){
-// For cases like 'first argument'
-msg=`The ${name}${determiner}${oneOf(expected,'type')}`;
-}else{
-consttype=name.includes('.') ? 'property' : 'argument';
-msg=`The "${name}" ${type}${determiner}${oneOf(expected,'type')}`;
-}
-
-// TODO(BridgeAR): Improve the output by showing `null` and similar.
-msg+=`. Received type ${typeofactual}`;
-returnmsg;
-}
-
-functionmissingArgs(...args){
-assert(args.length>0,'At least one arg needs to be specified');
-letmsg='The ';
-constlen=args.length;
-args=args.map((a)=>`"${a}"`);
-switch(len){
-case1:
-msg+=`${args[0]} argument`;
-break;
-case2:
-msg+=`${args[0]} and ${args[1]} arguments`;
-break;
-default:
-msg+=args.slice(0,len-1).join(', ');
-msg+=`, and ${args[len-1]} arguments`;
-break;
-}
-return`${msg} must be specified`;
-}
-
-functiononeOf(expected,thing){
-assert(typeofthing==='string','`thing` has to be of type string');
-if(Array.isArray(expected)){
-constlen=expected.length;
-assert(len>0,
-'At least one expected value needs to be specified');
-expected=expected.map((i)=>String(i));
-if(len>2){
-return`one of ${thing}${expected.slice(0,len-1).join(', ')}, or `+
-expected[len-1];
-}elseif(len===2){
-return`one of ${thing}${expected[0]} or ${expected[1]}`;
-}else{
-return`of ${thing}${expected[0]}`;
-}
-}else{
-return`of ${thing}${String(expected)}`;
-}
-}
-
-// Using a default argument here is important so the argument is not counted
-// towards `Function#length`.
-functionbufferOutOfBounds(name=undefined){
-if(name){
-return`"${name}" is outside of buffer bounds`;
-}
-return'Attempt to write outside buffer bounds';
-}
-
-// Using a default argument here is important so the argument is not counted
-// towards `Function#length`.
-functioninvalidChar(name,field=undefined){
-letmsg=`Invalid character in ${name}`;
-if(field!==undefined){
-msg+=` ["${field}"]`;
-}
-returnmsg;
-}
-
-functionoutOfRange(name,range,value){
-letmsg=`The value of "${name}" is out of range.`;
-if(range!==undefined)msg+=` It must be ${range}.`;
-msg+=` Received ${value}`;
-returnmsg;
-}