From 903a07b80e61581555971d1d7150044790130821 Mon Sep 17 00:00:00 2001 From: Max Kellermann Date: Wed, 11 Mar 2009 09:20:33 +0100 Subject: [PATCH] player_thread: don't call dc_stop() twice In the "CANCEL" command handler, the decoder is stopped twice: first by player_dc_stop(), then by dc_stop(). Remove the latter. --- src/player_thread.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/src/player_thread.c b/src/player_thread.c index 35d403044..54c83c6b5 100644 --- a/src/player_thread.c +++ b/src/player_thread.c @@ -363,12 +363,10 @@ static void player_process_command(struct player *player) return; } - if (dc.pipe != NULL && dc.pipe != player->pipe) { + if (dc.pipe != NULL && dc.pipe != player->pipe) /* the decoder is already decoding the song - stop it and reset the position */ player_dc_stop(player); - dc_stop(&pc.notify); - } pc.next_song = NULL; player->queued = false;