command: don't restart player in the "status" command

Don't attempt to restart the player if it was stopped, but there were still songs left on the playlist. This looks like it has been a workaround for a bug which has been fixed long time ago.

command: don't restart player in the "status" command
5541015a · Max Kellermann · 699b3117 · 5541015a · 5541015a · 5541015a
Commit 5541015a authored Jan 21, 2009 by Max Kellermann
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 4 deletions

command.c src/command.c +0 -1

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

playlist.h src/playlist.h +0 -2

No files found.
--- a/src/command.c
+++ b/src/command.c
@@ -451,7 +451,6 @@ handle_status(struct client *client,
 	int updateJobId;
 	int song;

-	playPlaylistIfPlayerStopped();
 	switch (getPlayerState()) {
 	case PLAYER_STATE_STOP:
 		state = "stop";

--- a/src/playlist.c
+++ b/src/playlist.c
@@ -861,6 +861,8 @@ enum playlist_result playPlaylistById(int id, int stopOnError)
 	return playPlaylist(song, stopOnError);
 }

+static void playPlaylistIfPlayerStopped(void);
+
 void syncPlayerAndPlaylist(void)
 {
 	if (playlist_state != PLAYLIST_STATE_PLAY)
@@ -911,7 +913,7 @@ void nextSongInPlaylist(void)
 	}
 }

-void playPlaylistIfPlayerStopped(void)
+static void playPlaylistIfPlayerStopped(void)
 {
 	if (getPlayerState() == PLAYER_STATE_STOP) {
 		enum player_error error = getPlayerError();

--- a/src/playlist.h
+++ b/src/playlist.h
@@ -149,8 +149,6 @@ int getPlaylistLength(void);

 unsigned long getPlaylistVersion(void);

-void playPlaylistIfPlayerStopped(void);
-
 enum playlist_result seekSongInPlaylist(unsigned song, float seek_time);

 enum playlist_result seekSongInPlaylistById(unsigned id, float seek_time);