From 26f2dc39a736326671656d8c2868957e88d0787e Mon Sep 17 00:00:00 2001
From: Stefan Kondinski <kondinskis@gmail.com>
Date: Wed, 10 Nov 2021 16:30:24 +0100
Subject: [PATCH] removes unnecessary thread spawn

---
 Cargo.lock  |  2 +-
 Cargo.toml  |  2 +-
 README.md   |  2 +-
 src/main.rs | 56 +++++++++++++++++++++++++----------------------------
 4 files changed, 29 insertions(+), 33 deletions(-)

diff --git a/Cargo.lock b/Cargo.lock
index c41a6a8..7104325 100644
--- a/Cargo.lock
+++ b/Cargo.lock
@@ -85,7 +85,7 @@ dependencies = [
 
 [[package]]
 name = "cloud-flight"
-version = "0.1.1"
+version = "0.1.2"
 dependencies = [
  "clokwerk",
  "hidapi",
diff --git a/Cargo.toml b/Cargo.toml
index f93ffcc..0df5c33 100644
--- a/Cargo.toml
+++ b/Cargo.toml
@@ -1,6 +1,6 @@
 [package]
 name = "cloud-flight"
-version = "0.1.1"
+version = "0.1.2"
 edition = "2018"
 
 # See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html
diff --git a/README.md b/README.md
index 9677042..2a09561 100644
--- a/README.md
+++ b/README.md
@@ -29,7 +29,7 @@ Once created replug the wireless dongle.
 Download the latest binary from the releases and run it.
 
 ```console
-foo@bar:~$ curl -LO https://github.com/kondinskis/hyperx-cloud-flight/releases/download/0.1.0/cloud-flight_amd64
+foo@bar:~$ curl -LO https://github.com/kondinskis/hyperx-cloud-flight/releases/download/0.1.2/cloud-flight_amd64
 foo@bar:~$ chmod +x cloud-flight_amd64
 foo@bar:~$ ./cloud-flight_amd64
 ```
diff --git a/src/main.rs b/src/main.rs
index 9e71ae0..08933c8 100644
--- a/src/main.rs
+++ b/src/main.rs
@@ -1,7 +1,6 @@
 use clokwerk::{Scheduler, TimeUnits};
 use simple_logger::SimpleLogger;
 use std::sync::Arc;
-use std::thread;
 
 const BATTERY_REFRESH_ON_MUTE: bool = true;
 
@@ -16,36 +15,33 @@ fn main() {
 
     let mut scheduler = Scheduler::new();
 
-    let handle = thread::spawn(move || {
-        let s_cf = cf.clone();
-        scheduler.every(3.minutes()).run(move || {
-            s_cf.battery();
-        });
-
-        cf.battery();
-
-        loop {
-            scheduler.run_pending();
-            let event = cf.read();
-            match event {
-                cloud_flight::Event::BatteryCharging => (),
-                cloud_flight::Event::Battery { value: _ } => (),
-                cloud_flight::Event::VolumeUp => (),
-                cloud_flight::Event::VolumeDown => (),
-                cloud_flight::Event::Muted => {
-                    if BATTERY_REFRESH_ON_MUTE {
-                        cf.battery();
-                    }
+    let s_cf = cf.clone();
+    scheduler.every(3.minutes()).run(move || {
+        s_cf.battery();
+    });
+
+    cf.battery();
+
+    loop {
+        scheduler.run_pending();
+        let event = cf.read();
+        match event {
+            cloud_flight::Event::BatteryCharging => (),
+            cloud_flight::Event::Battery { value: _ } => (),
+            cloud_flight::Event::VolumeUp => (),
+            cloud_flight::Event::VolumeDown => (),
+            cloud_flight::Event::Muted => {
+                if BATTERY_REFRESH_ON_MUTE {
+                    cf.battery();
                 }
-                cloud_flight::Event::Unmuted => (),
-                cloud_flight::Event::PowerOff => (),
-                cloud_flight::Event::PowerOn => (),
-                cloud_flight::Event::Ignored => (),
-            };
-            if !matches!(event, cloud_flight::Event::Ignored) {
-                svc.update();
             }
+            cloud_flight::Event::Unmuted => (),
+            cloud_flight::Event::PowerOff => (),
+            cloud_flight::Event::PowerOn => (),
+            cloud_flight::Event::Ignored => (),
+        };
+        if !matches!(event, cloud_flight::Event::Ignored) {
+            svc.update();
         }
-    });
-    handle.join().unwrap();
+    }
 }