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: nodejs#20486
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Joyee Cheung <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>

Author: BridgeAR

lib/internal/errors.js

@@ -407,6 +407,26 @@ function isStackOverflowError(err) {
          err.message === maxStack_ErrorMessage;
 }
 
+function oneOf(expected, thing) {
+  assert(typeof thing === 'string', '`thing` has to be of type string');
+  if (Array.isArray(expected)) {
+    const len = 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];
+    } else if (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,
@@ -441,7 +461,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);
@@ -579,7 +607,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(typeof name === 'string', "'name' must be a string");
+
+    // determiner: 'must be' or 'must not be'
+    let determiner;
+    if (typeof expected === 'string' && expected.startsWith('not ')) {
+      determiner = 'must not be';
+      expected = expected.replace(/^not /, '');
+    } else {
+      determiner = 'must be';
+    }
+
+    let msg;
+    if (name.endsWith(' argument')) {
+      // For cases like 'first argument'
+      msg = `The ${name} ${determiner} ${oneOf(expected, 'type')}`;
+    } else {
+      const type = 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 ${typeof actual}`;
+    return msg;
+  }, TypeError);
 E('ERR_INVALID_ARG_VALUE', (name, value, reason = 'is invalid') => {
   let inspected = util.inspect(value);
   if (inspected.length > 128) {
@@ -591,7 +644,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) => {
+    let msg = `Invalid character in ${name}`;
+    if (field !== undefined) {
+      msg += ` ["${field}"]`;
+    }
+    return msg;
+  }, TypeError);
 E('ERR_INVALID_CURSOR_POS',
   'Cannot set cursor row without setting its column', TypeError);
 E('ERR_INVALID_FD',
@@ -640,7 +702,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');
+    let msg = 'The ';
+    const len = args.length;
+    args = args.map((a) => `"${a}"`);
+    switch (len) {
+      case 1:
+        msg += `${args[0]} argument`;
+        break;
+      case 2:
+        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.' +
@@ -661,7 +742,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) => {
+    let msg = `The value of "${name}" is out of range.`;
+    if (range !== undefined) msg += ` It must be ${range}.`;
+    msg += ` Received ${value}`;
+    return msg;
+  }, 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);
@@ -758,95 +845,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);
-
-function invalidArgType(name, expected, actual) {
-  assert(typeof name === 'string', "'name' must be a string");
-
-  // determiner: 'must be' or 'must not be'
-  let determiner;
-  if (typeof expected === 'string' && expected.startsWith('not ')) {
-    determiner = 'must not be';
-    expected = expected.replace(/^not /, '');
-  } else {
-    determiner = 'must be';
-  }
-
-  let msg;
-  if (name.endsWith(' argument')) {
-    // For cases like 'first argument'
-    msg = `The ${name} ${determiner} ${oneOf(expected, 'type')}`;
-  } else {
-    const type = 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 ${typeof actual}`;
-  return msg;
-}
-
-function missingArgs(...args) {
-  assert(args.length > 0, 'At least one arg needs to be specified');
-  let msg = 'The ';
-  const len = args.length;
-  args = args.map((a) => `"${a}"`);
-  switch (len) {
-    case 1:
-      msg += `${args[0]} argument`;
-      break;
-    case 2:
-      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`;
-}
-
-function oneOf(expected, thing) {
-  assert(typeof thing === 'string', '`thing` has to be of type string');
-  if (Array.isArray(expected)) {
-    const len = 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];
-    } else if (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`.
-function bufferOutOfBounds(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`.
-function invalidChar(name, field = undefined) {
-  let msg = `Invalid character in ${name}`;
-  if (field !== undefined) {
-    msg += ` ["${field}"]`;
-  }
-  return msg;
-}
-
-function outOfRange(name, range, value) {
-  let msg = `The value of "${name}" is out of range.`;
-  if (range !== undefined) msg += ` It must be ${range}.`;
-  msg += ` Received ${value}`;
-  return msg;
-}