small stylistic improvements

src/geodb.rs

@@ -60,8 +60,7 @@ async fn save_etag(
         .wrap_err("failed to get GeoDB ETag path")?;
     tokio::fs::write(&etag_file, etag).await.wrap_err_with(move || {
         format!("failed to write to file {}", etag_file.display())
-    })?;
-    Ok(())
+    })
 }
 
 async fn save_geodb(
@@ -137,10 +136,8 @@ pub async fn download_geodb(
     }
 
     if let Some(etag_value) = etag {
-        save_etag(etag_value).await.wrap_err("failed to save GeoDB ETag")?;
+        save_etag(etag_value).await.wrap_err("failed to save GeoDB ETag")
     } else {
-        remove_etag().await.wrap_err("failed to remove GeoDB ETag")?;
+        remove_etag().await.wrap_err("failed to remove GeoDB ETag")
     }
-
-    Ok(())
 }

src/main.rs

@@ -122,6 +122,5 @@ async fn main() -> color_eyre::Result<()> {
     log::info!("Thank you for using proxy-scraper-checker!");
     tx.send(event::Event::App(event::AppEvent::Done))?;
     drop(tx);
-    ui_task.await??;
-    Ok(())
+    ui_task.await?
 }

src/parsers.rs

@@ -16,5 +16,5 @@ pub fn parse_ipv4(s: &str) -> Option<String> {
     IPV4_REGEX
         .captures(s)
         .unwrap()
-        .map(|captures| String::from(captures.name("host").unwrap().as_str()))
+        .map(|captures| captures.name("host").unwrap().as_str().to_owned())
 }

src/ui/tui.rs

@@ -53,8 +53,10 @@ impl super::UI for Tui {
         tx: tokio::sync::mpsc::UnboundedSender<Event>,
         mut rx: tokio::sync::mpsc::UnboundedReceiver<Event>,
     ) -> color_eyre::Result<()> {
-        let tick_task = tokio::spawn(tick_event_listener(tx.clone()));
-        let crossterm_task = tokio::spawn(crossterm_event_listener(tx));
+        let mut join_set = tokio::task::JoinSet::new();
+        join_set.spawn(tick_event_listener(tx.clone()));
+        join_set.spawn(crossterm_event_listener(tx));
+
         let mut app_state = AppState::new();
         let logger_state = TuiWidgetState::default();
         while !matches!(app_state.mode, AppMode::Quit) {
@@ -69,10 +71,9 @@ impl super::UI for Tui {
             }
         }
         drop(rx);
-        tick_task.await.wrap_err("failed to spawn tui tick task")??;
-        crossterm_task
-            .await
-            .wrap_err("failed to spawn tui crossterm task")??;
+        while let Some(task) = join_set.join_next().await {
+            task.wrap_err("failed to join event listener task")??;
+        }
         Ok(())
     }
 }