Merge pull request #74 from EFanZh/fixes-issues-71-and-73

Fix #71

Author: sebcrozet

src/macros.rs

@@ -82,37 +82,39 @@ macro_rules! ulps_ne {
 #[macro_export]
 macro_rules! __assert_approx {
     ($eq:ident, $given:expr, $expected:expr) => {{
-        let (given, expected) = (&($given), &($expected));
-
-        assert!($eq!(*given, *expected),
+        match (&($given), &($expected)) {
+            (given, expected) => assert!(
+                $eq!(*given, *expected),
 "assert_{}!({}, {})
 
     left  = {:?}
     right = {:?}
 
 ",
-            stringify!($eq),
-            stringify!($given),
-            stringify!($expected),
-            given, expected,
-        );
+                stringify!($eq),
+                stringify!($given),
+                stringify!($expected),
+                given, expected,
+            ),
+        }
     }};
     ($eq:ident, $given:expr, $expected:expr, $($opt:ident = $val:expr),+) => {{
-        let (given, expected) = (&($given), &($expected));
-
-        assert!($eq!(*given, *expected, $($opt = $val),+),
+        match (&($given), &($expected)) {
+            (given, expected) => assert!(
+                $eq!(*given, *expected, $($opt = $val),+),
 "assert_{}!({}, {}, {})
 
     left  = {:?}
     right = {:?}
 
 ",
-            stringify!($eq),
-            stringify!($given),
-            stringify!($expected),
-            stringify!($($opt = $val),+),
-            given, expected,
-        );
+                stringify!($eq),
+                stringify!($given),
+                stringify!($expected),
+                stringify!($($opt = $val),+),
+                given, expected,
+            ),
+        }
     }};
 }
 

tests/macros.rs

@@ -96,3 +96,10 @@ fn test_ulps_ne_trailing_commas() {
     let _: bool = ulps_ne!(1.0, 1.0,);
     let _: bool = ulps_ne!(1.0, 1.0, epsilon = 1.0, max_ulps = 1,);
 }
+
+#[test]
+fn test_rvalue_arguments() {
+    assert_abs_diff_eq!(vec![0.0].as_slice(), vec![0.0].as_slice());
+    assert_relative_eq!(vec![0.0].as_slice(), vec![0.0].as_slice());
+    assert_ulps_eq!(vec![0.0].as_slice(), vec![0.0].as_slice());
+}