Filter out empty locs

src/bin/diffenator3.rs

@@ -118,19 +118,6 @@ fn main() {
     }
 
     // Location-specific tests
-
-    // let loc_name: String = if let Some(ref loc) = cli.location {
-    //     let _hack = font_a.set_location(loc);
-    //     let _hack = font_b.set_location(loc);
-    //     loc.clone()
-    // } else if let Some(ref inst) = cli.instance {
-    //     font_a.set_instance(inst).expect("Couldn't find instance");
-    //     font_b.set_instance(inst).expect("Couldn't find instance");
-    //     inst.clone()
-    // } else {
-    //     "default".into()
-    // };
-
     let settings: Vec<Setting> = generate_settings(&cli, &font_a, &font_b);
 
     result.locations = settings
@@ -145,6 +132,11 @@ fn main() {
         })
         .collect();
 
+    // If there's more than one, filter out the boring ones
+    if result.locations.len() > 1 {
+        result.locations.retain(|l| l.is_some());
+    }
+
     // Report back
     if cli.html {
         reporters::html::report(

src/reporters/mod.rs

@@ -25,7 +25,7 @@ pub struct LocationResult {
 }
 
 impl LocationResult {
-    fn is_some(&self) -> bool {
+    pub fn is_some(&self) -> bool {
         self.error.is_some()
             || !self.glyphs.is_empty()
             || (self.words.is_some() && self.words.as_ref().unwrap().is_something())

src/templates/script.js

@@ -86,6 +86,11 @@ $(function () {
   }
   cmapDiff_static_html();
   $('[data-toggle="tooltip"]').tooltip()
+  if (!report["locations"]) {
+	$("#title").html("<h4 class='mt-2'>No differences found</h4>");
+	$("#ui-nav").hide();
+	return;
+  }
 
   for (var [index, loc] of report["locations"].entries()) {
     var loc_nav = $(`<li class="nav-item">