From 2d696f46c3ba84b0b2925788de99979b861f6eff Mon Sep 17 00:00:00 2001 From: Max Kellermann Date: Tue, 22 Oct 2013 00:13:39 +0200 Subject: PlaylistEdit: compare Song URIs in DeleteSong() Fixes purging deleted songs from the queue after update. Fixes regression from commit e96779d. --- src/PlaylistEdit.cxx | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) (limited to 'src/PlaylistEdit.cxx') diff --git a/src/PlaylistEdit.cxx b/src/PlaylistEdit.cxx index d4515d647..ff2ad5baa 100644 --- a/src/PlaylistEdit.cxx +++ b/src/PlaylistEdit.cxx @@ -323,8 +323,7 @@ void playlist::DeleteSong(struct player_control &pc, const struct Song &song) { for (int i = queue.GetLength() - 1; i >= 0; --i) - // TODO: compare URI instead of pointer - if (&song == &queue.Get(i)) + if (SongEquals(song, queue.Get(i))) DeletePosition(pc, i); } -- cgit v1.2.3