From 436335e9a3251cbfe42848843e1acea33d22177c Mon Sep 17 00:00:00 2001 From: Max Kellermann Date: Mon, 8 Apr 2013 21:22:06 +0200 Subject: player_control: don't emit IDLE_PLAYER before audio format is known Eliminates one IDLE_PLAYER call in playlist_control, and add two new ones to player_thread. Fixes Mantis bug 3636. --- NEWS | 1 + src/player_control.c | 2 -- src/player_thread.c | 4 ++++ 3 files changed, 5 insertions(+), 2 deletions(-) diff --git a/NEWS b/NEWS index ea746b9c8..dae06def2 100644 --- a/NEWS +++ b/NEWS @@ -1,6 +1,7 @@ ver 0.17.4 (2013/??/??) * protocol: - allow to omit END in ranges (START:END) + - don't emit IDLE_PLAYER before audio format is known * decoder: - ffmpeg: support float planar audio (ffmpeg 1.1) - ffmpeg: fix AVFrame allocation diff --git a/src/player_control.c b/src/player_control.c index 90f616d77..b18639087 100644 --- a/src/player_control.c +++ b/src/player_control.c @@ -127,8 +127,6 @@ pc_play(struct player_control *pc, struct song *song) assert(pc->next_song == NULL); player_unlock(pc); - - idle_add(IDLE_PLAYER); } void diff --git a/src/player_thread.c b/src/player_thread.c index 593788caf..ac0b0579e 100644 --- a/src/player_thread.c +++ b/src/player_thread.c @@ -313,6 +313,8 @@ player_open_output(struct player *player) pc->state = PLAYER_STATE_PLAY; player_unlock(pc); + idle_add(IDLE_PLAYER); + return true; } else { player->output_open = false; @@ -375,6 +377,8 @@ player_check_decoder_startup(struct player *player) pc->audio_format = dc->in_audio_format; player_unlock(pc); + idle_add(IDLE_PLAYER); + player->play_audio_format = dc->out_audio_format; player->decoder_starting = false; -- cgit v1.2.3