From b8baeb62261d26f30af7fde13dd1540e39c49626 Mon Sep 17 00:00:00 2001 From: Tarasov Aleksandr <55220741+arabianq@users.noreply.github.com> Date: Sat, 21 Mar 2026 21:03:30 +0300 Subject: [PATCH] refactor: replace contains_key/unwrap with if let Some in pipewire.rs (#39) This commit replaces two instances of `contains_key` followed by `get_mut().unwrap()` with the more idiomatic `if let Some(...)` pattern in `src/utils/pipewire.rs`. This reduces redundant hash map lookups and eliminates potential panics from `unwrap()`. Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com> --- src/utils/pipewire.rs | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/src/utils/pipewire.rs b/src/utils/pipewire.rs index fe0b21c..cc75331 100644 --- a/src/utils/pipewire.rs +++ b/src/utils/pipewire.rs @@ -153,8 +153,7 @@ pub async fn get_all_devices() -> Result<(Vec, Vec), B for port in ports { let node_id = port.node_id; - if input_devices.contains_key(&node_id) { - let input_device = input_devices.get_mut(&node_id).unwrap(); + if let Some(input_device) = input_devices.get_mut(&node_id) { match port.name.as_str() { "input_FL" => input_device.input_fl = Some(port), "input_FR" => input_device.input_fr = Some(port), @@ -172,8 +171,7 @@ pub async fn get_all_devices() -> Result<(Vec, Vec), B } _ => {} } - } else if output_devices.contains_key(&node_id) { - let output_device = output_devices.get_mut(&node_id).unwrap(); + } else if let Some(output_device) = output_devices.get_mut(&node_id) { match port.name.as_str() { "input_FL" => output_device.input_fl = Some(port), "input_FR" => output_device.input_fr = Some(port),