Merge pull request #46 from lonesometraveler/refactor_1

hacknus · web-flow · commit ad8bdd9afff4 · 2023-02-28T13:07:54.000+01:00
Improve clarity by directly calling `devices.contains(&amp;device)`
diff --git a/src/serial.rs b/src/serial.rs
@@ -133,8 +133,6 @@ pub fn serial_thread(
                 }
             }
 
-            let dev_is_con = devices.contains(&device);
-
             if reconnect {
                 print_to_console(
                     &print_lock,
@@ -146,7 +144,7 @@ pub fn serial_thread(
                 break 'connected_loop;
             }
 
-            if !dev_is_con {
+            if !devices.contains(&device) {
                 print_to_console(
                     &print_lock,
                     Print::Error(format!(

Original file line number	Diff line number	Diff line change
`@@ -133,8 +133,6 @@ pub fn serial_thread(`
`133`	`133`	`}`
`134`	`134`	`}`
`135`	`135`
`136`		`- let dev_is_con = devices.contains(&device);`
`137`		`-`
`138`	`136`	`if reconnect {`
`139`	`137`	`print_to_console(`
`140`	`138`	`&print_lock,`
`@@ -146,7 +144,7 @@ pub fn serial_thread(`
`146`	`144`	`break 'connected_loop;`
`147`	`145`	`}`
`148`	`146`
`149`		`- if !dev_is_con {`
	`147`	`+ if !devices.contains(&device) {`
`150`	`148`	`print_to_console(`
`151`	`149`	`&print_lock,`
`152`	`150`	`Print::Error(format!(`