From b978126bb0b22631306b095c7c6f4b2fdafe5557 Mon Sep 17 00:00:00 2001 From: Max Kellermann Date: Wed, 4 Dec 2013 15:11:23 +0100 Subject: DirectorySave: don't duplicate string Not necessary anymore for playlist_metadata_load(). --- src/DirectorySave.cxx | 14 +++----------- 1 file changed, 3 insertions(+), 11 deletions(-) (limited to 'src') diff --git a/src/DirectorySave.cxx b/src/DirectorySave.cxx index 6da961926..507f5ba47 100644 --- a/src/DirectorySave.cxx +++ b/src/DirectorySave.cxx @@ -29,7 +29,7 @@ #include "util/Error.hxx" #include "util/Domain.hxx" -#include +#include #define DIRECTORY_DIR "directory: " #define DIRECTORY_MTIME "mtime: " @@ -146,18 +146,10 @@ directory_load(TextFile &file, Directory &directory, Error &error) directory.AddSong(song); } else if (StringStartsWith(line, PLAYLIST_META_BEGIN)) { - /* duplicate the name, because - playlist_metadata_load() will overwrite the - buffer */ - char *name = g_strdup(line + sizeof(PLAYLIST_META_BEGIN) - 1); - + const char *name = line + sizeof(PLAYLIST_META_BEGIN) - 1; if (!playlist_metadata_load(file, directory.playlists, - name, error)) { - g_free(name); + name, error)) return false; - } - - g_free(name); } else { error.Format(directory_domain, "Malformed line: %s", line); -- cgit v1.2.3