Silence clippy, fmt, and add missing inline directives

Author: orlp

src/lib.rs

@@ -119,18 +119,22 @@ impl<T: Float> PartialOrd for OrderedFloat<T> {
         Some(self.cmp(other))
     }
 
+    #[inline]
     fn lt(&self, other: &Self) -> bool {
-        !(self >= other)
+        !self.ge(other)
     }
 
+    #[inline]
     fn le(&self, other: &Self) -> bool {
-        other >= self
+        other.ge(self)
     }
 
+    #[inline]
     fn gt(&self, other: &Self) -> bool {
-        !(other >= self)
+        !other.ge(self)
     }
 
+    #[inline]
     fn ge(&self, other: &Self) -> bool {
         // We consider all NaNs equal, and NaN is the largest possible
         // value. Thus if self is NaN we always return true. Otherwise
@@ -142,6 +146,8 @@ impl<T: Float> PartialOrd for OrderedFloat<T> {
 }
 
 impl<T: Float> Ord for OrderedFloat<T> {
+    #[inline]
+    #[allow(clippy::comparison_chain)]
     fn cmp(&self, other: &Self) -> Ordering {
         if self < other {
             Ordering::Less

tests/test.rs

@@ -27,13 +27,14 @@ fn test_total_order() {
     let numberline = [
         (-f32::INFINITY, 0),
         (-1.0, 1),
-        (-0.0, 2), (0.0, 2),
+        (-0.0, 2),
+        (0.0, 2),
         (1.0, 3),
         (f32::INFINITY, 4),
         (f32::NAN, 5),
         (-f32::NAN, 5),
     ];
-    
+
     for &(fi, i) in &numberline {
         for &(fj, j) in &numberline {
             assert_eq!(OrderedFloat(fi) < OrderedFloat(fj), i < j);