From bc85e92a722d945d193c9f3881e4ee9f96c8f2b6 Mon Sep 17 00:00:00 2001 From: Max Kellermann Date: Mon, 13 Oct 2008 16:30:44 +0200 Subject: playlist: don't use isPlaylist() in deletePlaylist() The only caller of deletePlaylist() appends PLAYLIST_FILE_SUFFIX, so we can be sure it's already there. We don't need to stat the file, since unlink() does all the checking. --- src/playlist.c | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) (limited to 'src/playlist.c') diff --git a/src/playlist.c b/src/playlist.c index 840c9c6b5..34a2311aa 100644 --- a/src/playlist.c +++ b/src/playlist.c @@ -1185,11 +1185,10 @@ enum playlist_result deletePlaylist(const char *utf8file) utf8_to_fs_playlist_path(path_max_tmp, utf8file); - if (!isPlaylist(path_max_tmp)) - return PLAYLIST_RESULT_NO_SUCH_LIST; - if (unlink(path_max_tmp) < 0) - return PLAYLIST_RESULT_ERRNO; + return errno == ENOENT + ? PLAYLIST_RESULT_NO_SUCH_LIST + : PLAYLIST_RESULT_ERRNO; return PLAYLIST_RESULT_SUCCESS; } -- cgit v1.2.3