---
I'm unsure if this is really correct, I don't think I understand
everything about the playlist handling, so see it as RFC.

I've produced a set of xspf playlists, and these wouldn't load from my
playlist directory, looking into the code I saw this function, which
checks for PLAYLIST_FILE_SUFFIX - which is m3u.

With the patch, which simply keeps the orignal name, my xspf playlists
load correctly.

// Simon

 src/stored_playlist.c |    8 +-------
 1 files changed, 1 insertions(+), 7 deletions(-)

diff --git a/src/stored_playlist.c b/src/stored_playlist.c
index 9b3ec4a..2284951 100644
--- a/src/stored_playlist.c
+++ b/src/stored_playlist.c
@@ -83,19 +83,13 @@ load_playlist_info(const char *parent_path_fs, const char 
*name_fs)
            memchr(name_fs, '\n', name_length) != NULL)
                return NULL;
 
-       if (!g_str_has_suffix(name_fs, PLAYLIST_FILE_SUFFIX))
-               return NULL;
-
        path_fs = g_build_filename(parent_path_fs, name_fs, NULL);
        ret = stat(path_fs, &st);
        g_free(path_fs);
        if (ret < 0 || !S_ISREG(st.st_mode))
                return NULL;
 
-       name = g_strndup(name_fs,
-                        name_length + 1 - sizeof(PLAYLIST_FILE_SUFFIX));
-       name_utf8 = fs_charset_to_utf8(name);
-       g_free(name);
+       name_utf8 = fs_charset_to_utf8(name_fs);
        if (name_utf8 == NULL)
                return NULL;
 
-- 
1.7.1


------------------------------------------------------------------------------
Enable your software for Intel(R) Active Management Technology to meet the
growing manageability and security demands of your customers. Businesses
are taking advantage of Intel(R) vPro (TM) technology - will your software 
be a part of the solution? Download the Intel(R) Manageability Checker 
today! http://p.sf.net/sfu/intel-dev2devmar
_______________________________________________
Musicpd-dev-team mailing list
Musicpd-dev-team@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/musicpd-dev-team

Reply via email to