From 4db51eeccd97bfbda1c7e6dd6bde41b344d7598d Mon Sep 17 00:00:00 2001 From: Max Kellermann Date: Wed, 4 Feb 2009 19:44:32 +0100 Subject: playlist: call syncPlaylistWithQueue() only in the event handler Don't call syncPlaylistWithQueue() in nextSongInPlaylist() and previousSongInPlaylist(). This is a relic from the time when there was no event, and was a workaround to the timing problem. --- src/playlist.c | 4 ---- 1 file changed, 4 deletions(-) (limited to 'src/playlist.c') diff --git a/src/playlist.c b/src/playlist.c index 0a32ce98b..7f44823c1 100644 --- a/src/playlist.c +++ b/src/playlist.c @@ -608,8 +608,6 @@ nextSongInPlaylist(struct playlist *playlist) assert(!queue_is_empty(&playlist->queue)); assert(queue_valid_order(&playlist->queue, playlist->current)); - syncPlaylistWithQueue(playlist); - playlist->stop_on_error = false; /* determine the next song from the queue's order list */ @@ -819,8 +817,6 @@ void previousSongInPlaylist(struct playlist *playlist) if (!playlist->playing) return; - syncPlaylistWithQueue(playlist); - if (diff && getPlayerElapsedTime() > PLAYLIST_PREV_UNLESS_ELAPSED) { /* re-start playing the current song (just like the "prev" button on CD players) */ -- cgit v1.2.3