aboutsummaryrefslogtreecommitdiffstats
path: root/src/song.c
diff options
context:
space:
mode:
authorEric Wong <normalperson@yhbt.net>2008-10-06 18:43:10 +0200
committerMax Kellermann <max@duempel.org>2008-10-06 18:43:10 +0200
commit43761441c9bbb763ccde3ae2f7f5508f5cec70ec (patch)
tree3d1bb799bf8bc6f78cbf96625f1a6d0a7204227d /src/song.c
parentdc245f389cffb1cb50d7dc066827be702b424a01 (diff)
downloadmpd-43761441c9bbb763ccde3ae2f7f5508f5cec70ec.tar.gz
mpd-43761441c9bbb763ccde3ae2f7f5508f5cec70ec.tar.xz
mpd-43761441c9bbb763ccde3ae2f7f5508f5cec70ec.zip
song: get rid of newNullSong()
It didn't save us any lines of code nor did it do anything useful since we would overwrite everything anyways.
Diffstat (limited to 'src/song.c')
-rw-r--r--src/song.c15
1 files changed, 2 insertions, 13 deletions
diff --git a/src/song.c b/src/song.c
index 9436df97c..75dd0257f 100644
--- a/src/song.c
+++ b/src/song.c
@@ -28,17 +28,6 @@
#include "os_compat.h"
-Song *newNullSong(void)
-{
- Song *song = xmalloc(sizeof(Song));
-
- song->tag = NULL;
- song->url = NULL;
- song->type = SONG_TYPE_FILE;
- song->parentDir = NULL;
-
- return song;
-}
Song *newSong(const char *url, enum song_type type, Directory * parentDir)
{
@@ -49,8 +38,8 @@ Song *newSong(const char *url, enum song_type type, Directory * parentDir)
return NULL;
}
- song = newNullSong();
-
+ song = xmalloc(sizeof(*song));
+ song->tag = NULL;
song->url = xstrdup(url);
song->type = type;
song->parentDir = parentDir;