git.delta.rocks / jrsonnet / refs/commits / 414a5eb5193d

difftreelog

feat remove clamp builtin

Petr Portnov2022-11-26parent: #ff64963.patch.diff
in: master

2 files changed

modifiedcrates/jrsonnet-stdlib/src/lib.rsdiffbeforeafterboth
81 ("sign", builtin_sign::INST),81 ("sign", builtin_sign::INST),
82 ("max", builtin_max::INST),82 ("max", builtin_max::INST),
83 ("min", builtin_min::INST),83 ("min", builtin_min::INST),
84 ("clamp", builtin_clamp::INST),
85 ("modulo", builtin_modulo::INST),84 ("modulo", builtin_modulo::INST),
86 ("floor", builtin_floor::INST),85 ("floor", builtin_floor::INST),
87 ("ceil", builtin_ceil::INST),86 ("ceil", builtin_ceil::INST),
modifiedcrates/jrsonnet-stdlib/src/math.rsdiffbeforeafterboth
--- a/crates/jrsonnet-stdlib/src/math.rs
+++ b/crates/jrsonnet-stdlib/src/math.rs
@@ -21,22 +21,6 @@
 }
 
 #[builtin]
-pub fn builtin_clamp(x: f64, min_val: f64, max_val: f64) -> Result<f64> {
-	debug_assert!(x.is_finite(), "jsonnet number are always finite");
-	debug_assert!(min_val.is_finite(), "jsonnet number are always finite");
-	debug_assert!(max_val.is_finite(), "jsonnet number are always finite");
-
-	// `f64::clamp` should noe be used here since it requires extra checks to guarantee NaN-safety
-	Ok(if x < min_val {
-		min_val
-	} else if x > max_val {
-		max_val
-	} else {
-		x
-	})
-}
-
-#[builtin]
 pub fn builtin_modulo(a: f64, b: f64) -> Result<f64> {
 	Ok(a % b)
 }