From 2c9a8229dae7ad9c7e6817f55137bd54d3d5726e Mon Sep 17 00:00:00 2001 From: Tal <83217276+talwat@users.noreply.github.com> Date: Tue, 5 Aug 2025 21:31:23 +0200 Subject: [PATCH] chore: make track list error more explicit --- src/player.rs | 9 +++++---- src/tracks/list.rs | 1 - 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/src/player.rs b/src/player.rs index 1fabd1e..2a08427 100644 --- a/src/player.rs +++ b/src/player.rs @@ -13,6 +13,7 @@ use std::{ use arc_swap::ArcSwapOption; use downloader::Downloader; +use eyre::Context; use reqwest::Client; use rodio::{OutputStream, OutputStreamBuilder, Sink}; use tokio::{ @@ -29,7 +30,7 @@ use mpris_server::{PlaybackStatus, PlayerInterface, Property}; use crate::{ messages::Messages, - play::PersistentVolume, + play::{PersistentVolume}, tracks::{self, list::List}, Args, }; @@ -108,7 +109,7 @@ impl Player { let volume = PersistentVolume::load().await?; // Load the track list. - let list = List::load(args.track_list.as_ref()).await?; + let list = List::load(args.track_list.as_ref()).await.wrap_err("unable to load the track list")?; // We should only shut up alsa forcefully on Linux if we really have to. #[cfg(target_os = "linux")] @@ -117,10 +118,10 @@ impl Player { } else { OutputStreamBuilder::open_default_stream()? }; - + stream.log_on_drop(false); // Frankly, this is a stupid feature. Stop shoving your crap into my beloved stderr!!! let sink = Sink::connect_new(stream.mixer()); - + if args.paused { sink.pause(); } diff --git a/src/tracks/list.rs b/src/tracks/list.rs index 8de291f..ed37192 100644 --- a/src/tracks/list.rs +++ b/src/tracks/list.rs @@ -127,7 +127,6 @@ impl List { if let Some(arg) = tracks { // Check if the track is in ~/.local/share/lowfi, in which case we'll load that. let name = data_dir()?.join(format!("{arg}.txt")); - let name = if name.exists() { name } else { arg.into() }; let raw = fs::read_to_string(name.clone()).await?;