From 3fff58fd5f05ccc92a6974ecd768723f848b88f9 Mon Sep 17 00:00:00 2001 From: Radiicall Date: Sat, 20 Apr 2024 09:03:53 +0200 Subject: [PATCH] cargo clippy --- jellyfin-rpc-cli/src/main.rs | 2 +- jellyfin-rpc/src/services/jellyfin.rs | 8 +++----- jellyfin-rpc/src/tests.rs | 4 ++-- 3 files changed, 6 insertions(+), 8 deletions(-) diff --git a/jellyfin-rpc-cli/src/main.rs b/jellyfin-rpc-cli/src/main.rs index 742d972..98fa659 100644 --- a/jellyfin-rpc-cli/src/main.rs +++ b/jellyfin-rpc-cli/src/main.rs @@ -59,7 +59,7 @@ async fn main() -> Result<(), Box> { if std::env::var("RUST_LOG").is_err() { let _ = tokio::task::spawn_blocking(move || { - std::env::set_var("RUST_LOG", &args.log_level); + std::env::set_var("RUST_LOG", args.log_level); }) .await; } diff --git a/jellyfin-rpc/src/services/jellyfin.rs b/jellyfin-rpc/src/services/jellyfin.rs index 4360e20..36b1077 100644 --- a/jellyfin-rpc/src/services/jellyfin.rs +++ b/jellyfin-rpc/src/services/jellyfin.rs @@ -259,12 +259,10 @@ impl Content { } else { state = format!("S{:02}E{:02}", season, first_episode_number); } + } else if config.jellyfin.add_divider.unwrap_or(false) { + state = format!("S{} - E{}", season, first_episode_number); } else { - if config.jellyfin.add_divider.unwrap_or(false) { - state = format!("S{} - E{}", season, first_episode_number); - } else { - state = format!("S{}E{}", season, first_episode_number); - } + state = format!("S{}E{}", season, first_episode_number); }; // Does this if statement work? diff --git a/jellyfin-rpc/src/tests.rs b/jellyfin-rpc/src/tests.rs index d316ccf..f046350 100644 --- a/jellyfin-rpc/src/tests.rs +++ b/jellyfin-rpc/src/tests.rs @@ -109,7 +109,7 @@ fn media_type_is_none() { let media_type_2 = MediaType::None; assert_eq!( - media_type_1.is_none() == false, - media_type_2.is_none() == true + !media_type_1.is_none(), + media_type_2.is_none() ) }