Math.clamp: remove a RangeError if min <= max or +0 min and -0 max

Author: zloirock

CHANGELOG.md

@@ -37,6 +37,7 @@
   - Built-ins:
     - `Math.clamp`
   - Moved to stage 2, [May 2025 TC39 meeting](https://github.com/tc39/proposal-math-clamp/commit/a005f28a6a03e175b9671de1c8c70dd5b7b08c2d)
+  - Removed a `RangeError` if `min <= max` or `+0` min and `-0` max, [tc39/proposal-math-clamp/#22](https://github.com/tc39/proposal-math-clamp/issues/22)
 - Always check regular expression flags by `flags` getter [PR](https://github.com/tc39/ecma262/pull/2791). Native methods are not fixed, only own implementation updated for:
   - `RegExp.prototype[@@match]`
   - `RegExp.prototype[@@replace]`

packages/core-js/internals/math-clamp.js

@@ -0,0 +1,13 @@
+'use strict';
+var aNumber = require('../internals/a-number');
+var notANaN = require('../internals/not-a-nan');
+
+var $min = Math.min;
+var $max = Math.max;
+
+module.exports = function clamp(value, min, max) {
+  aNumber(value);
+  notANaN(aNumber(min));
+  notANaN(aNumber(max));
+  return $min(max, $max(min, value));
+};

packages/core-js/modules/esnext.math.clamp.js

@@ -1,21 +1,9 @@
 'use strict';
 var $ = require('../internals/export');
-var aNumber = require('../internals/a-number');
-var notANaN = require('../internals/not-a-nan');
-var sameValue = require('../internals/same-value');
-
-var $RangeError = RangeError;
-var $min = Math.min;
-var $max = Math.max;
+var clamp = require('../internals/math-clamp');
 
 // `Math.clamp` method
 // https://github.com/tc39/proposal-math-clamp
 $({ target: 'Math', stat: true, forced: true }, {
-  clamp: function clamp(value, min, max) {
-    aNumber(value);
-    notANaN(aNumber(min));
-    notANaN(aNumber(max));
-    if ((sameValue(min, 0) && sameValue(max, -0)) || min > max) throw new $RangeError('`min` should be smaller than `max`');
-    return $min(max, $max(min, value));
-  }
+  clamp: clamp
 });

tests/unit-global/esnext.math.clamp.js

@@ -17,11 +17,12 @@ QUnit.test('Math.clamp', assert => {
   assert.same(clamp(0, -1, -0), -0, 'If value is +0𝔽 and max is -0𝔽, return -0𝔽.');
   assert.same(clamp(0, -0, -0), -0, 'If min = max return min.');
 
+  assert.same(clamp(2, 0, -0), -0, 'min is +0𝔽 and max is -0𝔽');
+  assert.same(clamp(2, 3, 1), 1, 'min > max');
+
   assert.throws(() => clamp(Object(2), 1, 3), TypeError, 'If value is not a Number, throw a TypeError exception.');
   assert.throws(() => clamp(2, Object(1), 3), TypeError, 'If min is not a Number, throw a TypeError exception.');
   assert.throws(() => clamp(2, NaN, 3), RangeError, 'If min is NaN, throw a RangeError exception.');
   assert.throws(() => clamp(2, 1, Object(3)), TypeError, 'If max is not a Number, throw a TypeError exception.');
   assert.throws(() => clamp(2, 1, NaN), RangeError, 'If max is NaN, throw a RangeError exception.');
-  assert.throws(() => clamp(2, 0, -0), RangeError, 'If min is +0𝔽 and max is -0𝔽, throw a RangeError exception.');
-  assert.throws(() => clamp(2, 3, 1), RangeError, 'If min > max, throw a RangeError exception.');
 });

tests/unit-pure/esnext.math.clamp.js

@@ -15,11 +15,12 @@ QUnit.test('Math.clamp', assert => {
   assert.same(clamp(0, -1, -0), -0, 'If value is +0𝔽 and max is -0𝔽, return -0𝔽.');
   assert.same(clamp(0, -0, -0), -0, 'If min = max return min.');
 
+  assert.same(clamp(2, 0, -0), -0, 'min is +0𝔽 and max is -0𝔽');
+  assert.same(clamp(2, 3, 1), 1, 'min > max');
+
   assert.throws(() => clamp(Object(2), 1, 3), TypeError, 'If value is not a Number, throw a TypeError exception.');
   assert.throws(() => clamp(2, Object(1), 3), TypeError, 'If min is not a Number, throw a TypeError exception.');
   assert.throws(() => clamp(2, NaN, 3), RangeError, 'If min is NaN, throw a RangeError exception.');
   assert.throws(() => clamp(2, 1, Object(3)), TypeError, 'If max is not a Number, throw a TypeError exception.');
   assert.throws(() => clamp(2, 1, NaN), RangeError, 'If max is NaN, throw a RangeError exception.');
-  assert.throws(() => clamp(2, 0, -0), RangeError, 'If min is +0𝔽 and max is -0𝔽, throw a RangeError exception.');
-  assert.throws(() => clamp(2, 3, 1), RangeError, 'If min > max, throw a RangeError exception.');
 });