Cargo clippy fix

Author: joelsa

src/main.rs

@@ -139,11 +139,10 @@ async fn main() -> Result<()> {
         let shutdown_tx = shutdown_tx.clone();
         ctrlc::set_handler(move || {
             running.store(false, Ordering::SeqCst);
-            if let Ok(tx_guard) = shutdown_tx.lock() {
-                if let Some(tx) = tx_guard.as_ref() {
+            if let Ok(tx_guard) = shutdown_tx.lock()
+                && let Some(tx) = tx_guard.as_ref() {
                     let _ = tx.send(UiMessage::Quit);
                 }
-            }
         })
         .expect("Failed to set Ctrl-C handler");
     }

src/serial_io.rs

@@ -84,7 +84,7 @@ impl SerialReader {
         let mut hex_str = String::with_capacity(capacity);
 
         if self.log_ts {
-            hex_str.push_str("[");
+            hex_str.push('[');
             hex_str.push_str(&Utc::now().format("%Y-%m-%d %H:%M:%S%.3f").to_string());
             hex_str.push_str("] ");
         }
@@ -105,7 +105,7 @@ impl SerialReader {
         let mut text = String::with_capacity(capacity);
 
         if self.log_ts {
-            text.push_str("[");
+            text.push('[');
             text.push_str(&Utc::now().format("%Y-%m-%d %H:%M:%S%.3f").to_string());
             text.push_str("] ");
         }
@@ -116,8 +116,8 @@ impl SerialReader {
     }
 
     async fn write_to_log(&mut self, bytes: &[u8]) {
-        if let Some(w) = &self.rx_log_writer {
-            if let Ok(mut lw) = w.lock() {
+        if let Some(w) = &self.rx_log_writer
+            && let Ok(mut lw) = w.lock() {
                 use std::io::Write;
 
                 if self.log_ts {
@@ -136,7 +136,6 @@ impl SerialReader {
 
                 let _ = lw.flush();
             }
-        }
     }
 }
 

src/ui/mod.rs

@@ -144,32 +144,30 @@ async fn handle_enter_key(
     // Send the complete line to serial port
     if !input.is_empty() {
         write_bytes_async(port, input.as_bytes()).await?;
-        if let Some(w) = &ui_config.tx_log {
-            if let Ok(mut lw) = w.lock() {
+        if let Some(w) = &ui_config.tx_log
+            && let Ok(mut lw) = w.lock() {
                 use std::io::Write;
                 if ui_config.log_ts {
                     let _ = write!(lw, "[{}] ", Utc::now().format("%Y-%m-%d %H:%M:%S%.3f"));
                 }
                 let _ = lw.write_all(input.as_bytes());
                 let _ = lw.flush();
             }
-        }
     }
 
     // Send line ending
     let end = ui_config.line_ending.bytes();
     if !end.is_empty() {
         write_bytes_async(port, end).await?;
-        if let Some(w) = &ui_config.tx_log {
-            if let Ok(mut lw) = w.lock() {
+        if let Some(w) = &ui_config.tx_log
+            && let Ok(mut lw) = w.lock() {
                 use std::io::Write;
                 if ui_config.log_ts && input.is_empty() {
                     let _ = write!(lw, "[{}] ", Utc::now().format("%Y-%m-%d %H:%M:%S%.3f"));
                 }
                 let _ = lw.write_all(end);
                 let _ = lw.flush();
             }
-        }
     }
 
     Ok(())