playlist_control: "previous" really plays the previous song

No more CD player emulation.  The current behaviour of "previous" is
difficult for a client to predict, because it does not definitely know
the current position within the song.  If a client wants to restart
the current song, it can always send "playid".

NEWS

@@ -5,6 +5,7 @@ ver 0.16 (20??/??/??)
   - removed the deprecated "volume" command
   - added the "findadd" command
   - range support for "delete"
+  - "previous" really plays the previous song
 * input:
   - lastfm: use metadata
 * tags:

src/playlist.c

@@ -62,16 +62,12 @@ playlist_init(struct playlist *playlist)
 
 	playlist->queued = -1;
 	playlist->current = -1;
-
-	playlist->prev_elapsed = g_timer_new();
 }
 
 void
 playlist_finish(struct playlist *playlist)
 {
 	queue_finish(&playlist->queue);
-
-	g_timer_destroy(playlist->prev_elapsed);
 }
 
 /**

src/playlist.h

@@ -81,13 +81,6 @@ struct playlist {
 	 * This variable is only valid if #playing is true.
 	 */
 	int queued;
-
-	/**
-	 * This timer tracks the time elapsed since the last "prev"
-	 * command.  If that is less than one second ago, "prev" jumps
-	 * to the previous song instead of rewinding the current song.
-	 */
-	GTimer *prev_elapsed;
 };
 
 /** the global playlist object */

src/playlist_control.c

@@ -30,14 +30,6 @@
 #undef G_LOG_DOMAIN
 #define G_LOG_DOMAIN "playlist"
 
-enum {
-	/**
-	 * When the "prev" command is received, rewind the current
-	 * track if this number of seconds has already elapsed.
-	 */
-	PLAYLIST_PREV_UNLESS_ELAPSED = 10,
-};
-
 void playlist_stop(struct playlist *playlist)
 {
 	if (!playlist->playing)
@@ -191,13 +183,8 @@ void playlist_previous(struct playlist *playlist)
 	if (!playlist->playing)
 		return;
 
-	if (g_timer_elapsed(playlist->prev_elapsed, NULL) >= 1.0 &&
+	assert(queue_length(&playlist->queue) > 0);
-	    getPlayerElapsedTime() > PLAYLIST_PREV_UNLESS_ELAPSED) {
-		/* re-start playing the current song (just like the
-		   "prev" button on CD players) */
 
-		playlist_play_order(playlist, playlist->current);
-	} else {
 	if (playlist->current > 0) {
 		/* play the preceding song */
 		playlist_play_order(playlist,
@@ -213,9 +200,6 @@ void playlist_previous(struct playlist *playlist)
 	}
 }
 
-	g_timer_start(playlist->prev_elapsed);
-}
-
 enum playlist_result
 playlist_seek_song(struct playlist *playlist, unsigned song, float seek_time)
 {