From c04293d8b85116ea6f7189e8f5962888d2372f42 Mon Sep 17 00:00:00 2001
From: Igor null <m1el.2027@gmail.com>
Date: Mon, 1 Jul 2019 17:05:46 +0300
Subject: [PATCH 1/2] Fixed rounding to negative zero

---
 src/math/round.rs  | 15 ++++++++++-----
 src/math/roundf.rs | 11 ++++++++---
 2 files changed, 18 insertions(+), 8 deletions(-)

diff --git a/src/math/round.rs b/src/math/round.rs
index 9a9723c..efbe68a 100644
--- a/src/math/round.rs
+++ b/src/math/round.rs
@@ -5,20 +5,20 @@ const TOINT: f64 = 1.0 / f64::EPSILON;
 #[inline]
 #[cfg_attr(all(test, assert_no_panic), no_panic::no_panic)]
 pub fn round(mut x: f64) -> f64 {
-    let (f, i) = (x, x.to_bits());
+    let i = x.to_bits();
     let e: u64 = i >> 52 & 0x7ff;
     let mut y: f64;
 
     if e >= 0x3ff + 52 {
         return x;
     }
-    if i >> 63 != 0 {
-        x = -x;
-    }
     if e < 0x3ff - 1 {
         // raise inexact if x!=0
         force_eval!(x + TOINT);
-        return 0.0 * f;
+        return 0.0 * x;
+    }
+    if i >> 63 != 0 {
+        x = -x;
     }
     y = x + TOINT - TOINT - x;
     if y > 0.5 {
@@ -35,3 +35,8 @@ pub fn round(mut x: f64) -> f64 {
         y
     }
 }
+
+#[test]
+fn negative_zero() {
+    assert_eq!(round(-0.0_f64).to_bits(), (-0.0_f64).to_bits());
+}
diff --git a/src/math/roundf.rs b/src/math/roundf.rs
index 839d946..559d3a9 100644
--- a/src/math/roundf.rs
+++ b/src/math/roundf.rs
@@ -12,13 +12,13 @@ pub fn roundf(mut x: f32) -> f32 {
     if e >= 0x7f + 23 {
         return x;
     }
-    if i >> 31 != 0 {
-        x = -x;
-    }
     if e < 0x7f - 1 {
         force_eval!(x + TOINT);
         return 0.0 * x;
     }
+    if i >> 31 != 0 {
+        x = -x;
+    }
     y = x + TOINT - TOINT - x;
     if y > 0.5f32 {
         y = y + x - 1.0;
@@ -33,3 +33,8 @@ pub fn roundf(mut x: f32) -> f32 {
         y
     }
 }
+
+#[test]
+fn negative_zero() {
+    assert_eq!(roundf(-0.0_f32).to_bits(), (-0.0_f32).to_bits());
+}

From 406bff1844250eb58eebf35e0e53e70003d5b9d8 Mon Sep 17 00:00:00 2001
From: Igor null <m1el.2027@gmail.com>
Date: Mon, 1 Jul 2019 17:23:52 +0300
Subject: [PATCH 2/2] move tests to separate #[cfg(test)] mod

---
 src/math/round.rs  | 11 ++++++++---
 src/math/roundf.rs | 11 ++++++++---
 2 files changed, 16 insertions(+), 6 deletions(-)

diff --git a/src/math/round.rs b/src/math/round.rs
index efbe68a..67590d2 100644
--- a/src/math/round.rs
+++ b/src/math/round.rs
@@ -36,7 +36,12 @@ pub fn round(mut x: f64) -> f64 {
     }
 }
 
-#[test]
-fn negative_zero() {
-    assert_eq!(round(-0.0_f64).to_bits(), (-0.0_f64).to_bits());
+#[cfg(test)]
+mod tests {
+    use super::round;
+
+    #[test]
+    fn negative_zero() {
+        assert_eq!(round(-0.0_f64).to_bits(), (-0.0_f64).to_bits());
+    }
 }
diff --git a/src/math/roundf.rs b/src/math/roundf.rs
index 559d3a9..85114be 100644
--- a/src/math/roundf.rs
+++ b/src/math/roundf.rs
@@ -34,7 +34,12 @@ pub fn roundf(mut x: f32) -> f32 {
     }
 }
 
-#[test]
-fn negative_zero() {
-    assert_eq!(roundf(-0.0_f32).to_bits(), (-0.0_f32).to_bits());
+#[cfg(test)]
+mod tests {
+    use super::roundf;
+
+    #[test]
+    fn negative_zero() {
+        assert_eq!(roundf(-0.0_f32).to_bits(), (-0.0_f32).to_bits());
+    }
 }