From d9f8ec54282d3163bae3dda41c42a55a69d92533 Mon Sep 17 00:00:00 2001 From: phoenix Date: Sat, 11 Oct 2025 15:38:34 -0400 Subject: [PATCH] Code cleanup --- src/song.rs | 13 +++---------- tests/tests.rs | 13 +++++-------- 2 files changed, 8 insertions(+), 18 deletions(-) diff --git a/src/song.rs b/src/song.rs index 600daa5..183d08b 100644 --- a/src/song.rs +++ b/src/song.rs @@ -114,25 +114,18 @@ impl Song { let p = std::path::Path::new(&song_path); if p.exists() { match std::fs::remove_file(&p) { - Ok(_) => { - Ok(()) - } - Err(err) => { - Err(err) - } + Ok(_) => Ok(()), + Err(err) => Err(err), } } else { Ok(()) } } - Err(err) => { - Err(err) - } + Err(err) => Err(err), } } } - /// Generates a filename. In order to save a song to the filesystem, the song must have /// a directory and filename pub fn generate_filename(typ: types::MusicTypes, randomize: bool) -> String { diff --git a/tests/tests.rs b/tests/tests.rs index a153ecb..386940b 100644 --- a/tests/tests.rs +++ b/tests/tests.rs @@ -132,15 +132,12 @@ mod song_tests { }; match song::io::copy_song(&song, &mut copied_song) { - Ok(_) => { - match copied_song.remove_from_filesystem() { - Ok(_) => { - } - Err(err) => { - assert!(false, "Error: {err:?}") - } + Ok(_) => match copied_song.remove_from_filesystem() { + Ok(_) => {} + Err(err) => { + assert!(false, "Error: {err:?}") } - } + }, Err(err) => { assert!(false, "Error: {err:?}") }