clippy:option_map_unit_fn

warning: called `map(f)` on an `Option` value where `f` is a closure that returns the unit type `()`
    --> src/cmd/geocode.rs:1379:5
     |
1379 | //     engine.metadata.as_ref().map(|m| {
1380 | ||         let now = std::time::SystemTime::now();
1381 | ||         let age = now.duration_since(m.created_at).unwrap();
1382 | ||         progressbar.println(format!(
...    ||
1385 | ||         ));
1386 | ||     });
     | ||______^- help: try: `if let Some(m) = engine.metadata.as_ref() { ... }`
     |  |______|
     |
     |
     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#option_map_unit_fn
     = note: `#[warn(clippy::option_map_unit_fn)]` on by default

warning: `qsv` (bin "qsv") generated 1 warning

src/cmd/geocode.rs

@@ -1376,14 +1376,14 @@ async fn load_engine(geocode_index_file: PathBuf, progressbar: &ProgressBar) ->
         .load_from(geocode_index_file)
         .map_err(|e| format!("On load index file: {e}"))?;
 
-    engine.metadata.as_ref().map(|m| {
+    if let Some(metadata) = &engine.metadata {
         let now = std::time::SystemTime::now();
-        let age = now.duration_since(m.created_at).unwrap();
+        let age = now.duration_since(metadata.created_at).unwrap();
         progressbar.println(format!(
             "Geonames index loaded. Age: {}",
             indicatif::HumanDuration(age)
         ));
-    });
+    }
 
     Ok(engine)
 }