Removed Numeric separator (#491)

nodejs · Jan 1, 2023 · f135e4b · f135e4b
1 parent b733ae5
commit f135e4b
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/build/replacements.mjs b/build/replacements.mjs
@@ -91,6 +91,8 @@ const internalValidatorsRequireRelativeUtil = ["require\\('internal/util'\\)", "
 
 const internalValidatorsRequireUtilTypes = ["require\\('internal/util/types'\\)", "require('../ours/util').types"]
 
+const internalValidatorsNumericSeparator = ['4_294_967_295', '4294967295']
+
 const streamIndexIsUint8Array = [
   "Stream._isUint8Array = require\\('internal/util/types'\\).isUint8Array;",
   `
@@ -253,7 +255,8 @@ export const replacements = {
     internalValidatorsRequireRelativeUtil,
     internalValidatorsRequireUtilTypes,
     internalValidatorsNoRequireSignals,
-    internalValidatorsNoCoalesceAssignment
+    internalValidatorsNoCoalesceAssignment,
+    internalValidatorsNumericSeparator
   ],
   'lib/stream.js': [
     streamIndexIsUint8Array,

diff --git a/lib/internal/validators.js b/lib/internal/validators.js
@@ -125,7 +125,7 @@ const validateUint32 = hideStackFrames((value, name, positive = false) => {
   }
   const min = positive ? 1 : 0
   // 2 ** 32 === 4294967296
-  const max = 4_294_967_295
+  const max = 4294967295
   if (value < min || value > max) {
     throw new ERR_OUT_OF_RANGE(name, `>= ${min} && <= ${max}`, value)
   }