fn constrain: Cleanup and make safe (#1134)

memorysafety · May 28, 2024 · adee7fd · adee7fd
2 parents f36b93b + 3f5de53
commit adee7fd
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/src/cdef.rs b/src/cdef.rs
@@ -539,12 +539,12 @@ extern "C" {
 }
 
 #[inline]
-pub unsafe fn constrain(diff: c_int, threshold: c_int, shift: c_int) -> c_int {
+pub fn constrain(diff: c_int, threshold: c_int, shift: c_int) -> c_int {
     let adiff = diff.abs();
-    return apply_sign(
-        cmp::min(adiff, cmp::max(0 as c_int, threshold - (adiff >> shift))),
+    apply_sign(
+        cmp::min(adiff, cmp::max(0, threshold - (adiff >> shift))),
         diff,
-    );
+    )
 }
 
 #[inline]