From ff650d50a21032c13c66f5514892276f67b2468a Mon Sep 17 00:00:00 2001 From: Max Kellermann Date: Fri, 31 Jan 2014 23:14:13 +0100 Subject: Mapper: assert that Song::parent is not nullptr --- src/Mapper.cxx | 21 +++------------------ 1 file changed, 3 insertions(+), 18 deletions(-) (limited to 'src') diff --git a/src/Mapper.cxx b/src/Mapper.cxx index 8d9544932..60928dbb9 100644 --- a/src/Mapper.cxx +++ b/src/Mapper.cxx @@ -225,21 +225,6 @@ map_directory_child_fs(const Directory &directory, const char *name) return AllocatedPath::Build(parent_fs, name_fs); } -/** - * Map a song object that was created by song_dup_detached(). It does - * not have a real parent directory, only the dummy object - * #detached_root. - */ -static AllocatedPath -map_detached_song_fs(const char *uri_utf8) -{ - auto uri_fs = AllocatedPath::FromUTF8(uri_utf8); - if (uri_fs.IsNull()) - return uri_fs; - - return AllocatedPath::Build(music_dir_fs, uri_fs); -} - DetachedSong map_song_detach(const LightSong &song) { @@ -257,9 +242,9 @@ map_song_detach(const LightSong &song) AllocatedPath map_song_fs(const Song &song) { - return song.parent == nullptr - ? map_detached_song_fs(song.uri) - : map_directory_child_fs(*song.parent, song.uri); + assert(song.parent != nullptr); + + return map_directory_child_fs(*song.parent, song.uri); } #endif -- cgit v1.2.3