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.

player_thread: don't call dc_stop() twice
903a07b8 · Max Kellermann · a7318a64 · 903a07b8
Commit 903a07b8 authored Mar 11, 2009 by Max Kellermann
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 3 deletions

player_thread.c src/player_thread.c +1 -3

No files found.
--- 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;