Fix ordering of NLL dashboard

Fixes rust-lang#273.

Author: simonlindholm

site/src/api.rs

@@ -173,9 +173,9 @@ pub mod nll_dashboard {
     }
 
     impl Point {
-        pub fn pct(&self) -> Option<u64> {
+        pub fn pct(&self) -> Option<f32> {
             if let (Some(clean), Some(nll)) = (self.clean, self.nll) {
-                Some((100.0 * nll / clean) as u64)
+                Some(100.0 * nll / clean)
             } else {
                 None
             }

site/src/server.rs

@@ -125,8 +125,9 @@ pub fn handle_nll_dashboard(
         }).collect::<Vec<_>>();
     points.sort_by(|a, b| {
         match (a.pct(), b.pct()) {
-            (Some(a), Some(b)) => a.cmp(&b).reverse(),
+            (Some(a), Some(b)) => a.partial_cmp(&b).unwrap_or(Ordering::Equal).reverse(),
             (Some(_), None) => Ordering::Less,
+            (None, Some(_)) => Ordering::Greater,
             _ => a.case.cmp(&b.case),
         }
     });