From 3a9697adf268d780500cc53432efa4f164db7bb4 Mon Sep 17 00:00:00 2001 From: Maarten Sebregts Date: Mon, 19 Dec 2011 21:15:00 +0100 Subject: Playlist: fix bug in moving after current song Moving songs using either 'move' or 'moveid' to position -1 (after the current song) would fail for a song which is just before the current song. This patch corrects the check to see if the current song is in the range to be moved. Since the range is from `start` up to `end` (exclusive) the check was incorrect, but is now fixed. --- src/playlist_edit.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src') diff --git a/src/playlist_edit.c b/src/playlist_edit.c index c54b72750..3bcb2ce14 100644 --- a/src/playlist_edit.c +++ b/src/playlist_edit.c @@ -356,7 +356,7 @@ playlist_move_range(struct playlist *playlist, playlist->current) : -1; if (to < 0 && playlist->current >= 0) { - if (start <= (unsigned)currentSong && (unsigned)currentSong <= end) + if (start <= (unsigned)currentSong && (unsigned)currentSong < end) /* no-op, can't be moved to offset of itself */ return PLAYLIST_RESULT_SUCCESS; to = (currentSong + abs(to)) % queue_length(&playlist->queue); -- cgit v1.2.3