player_control: eliminate PLAYER_COMMAND_PLAY

Sending PLAYER_COMMAND_STOP followed by PLAYER_COMMAND_QUEUE does the same. PLAYER_COMMAND_PLAY is redundant.

player_control: eliminate PLAYER_COMMAND_PLAY
d6a6f428 · Max Kellermann · fd3934b8 · d6a6f428 · d6a6f428 · d6a6f428
Commit d6a6f428 authored Oct 08, 2009 by Max Kellermann
Show whitespace changes
Inline Side-by-side

Showing with 1 addition and 5 deletions

player_control.c src/player_control.c +1 -2

player_control.h src/player_control.h +0 -1

player_thread.c src/player_thread.c +0 -2

No files found.
--- a/src/player_control.c
+++ b/src/player_control.c
@@ -77,8 +77,7 @@ pc_play(struct song *song)
 	assert(pc.next_song == NULL);
-	pc.next_song = song;
+	pc_enqueue_song(song);
-	player_command(PLAYER_COMMAND_PLAY);
 	assert(pc.next_song == NULL);

--- a/src/player_control.h
+++ b/src/player_control.h
@@ -35,7 +35,6 @@ enum player_command {
 	PLAYER_COMMAND_NONE = 0,
 	PLAYER_COMMAND_EXIT,
 	PLAYER_COMMAND_STOP,
-	PLAYER_COMMAND_PLAY,
 	PLAYER_COMMAND_PAUSE,
 	PLAYER_COMMAND_SEEK,
 	PLAYER_COMMAND_CLOSE_AUDIO,

--- a/src/player_thread.c
+++ b/src/player_thread.c
@@ -347,7 +347,6 @@ static void player_process_command(struct player *player)
 {
 	switch (pc.command) {
 	case PLAYER_COMMAND_NONE:
-	case PLAYER_COMMAND_PLAY:
 	case PLAYER_COMMAND_STOP:
 	case PLAYER_COMMAND_EXIT:
 	case PLAYER_COMMAND_CLOSE_AUDIO:
@@ -773,7 +772,6 @@ static gpointer player_task(G_GNUC_UNUSED gpointer arg)
 	while (1) {
 		switch (pc.command) {
-		case PLAYER_COMMAND_PLAY:
 		case PLAYER_COMMAND_QUEUE:
 			assert(pc.next_song != NULL);