aboutsummaryrefslogtreecommitdiffstats
path: root/src
diff options
context:
space:
mode:
authorJ. Alexander Treuman <jat@spatialrift.net>2007-05-26 14:09:09 +0000
committerJ. Alexander Treuman <jat@spatialrift.net>2007-05-26 14:09:09 +0000
commitf16e3728711c3dd4fd2ca241ddbe3df4499d754e (patch)
tree42f2e7f653eb65f5ea17df82ff7d1aa3e230829a /src
parentf4d959a07b8a52233430dbf3d461b565628795c3 (diff)
downloadmpd-f16e3728711c3dd4fd2ca241ddbe3df4499d754e.tar.gz
mpd-f16e3728711c3dd4fd2ca241ddbe3df4499d754e.tar.xz
mpd-f16e3728711c3dd4fd2ca241ddbe3df4499d754e.zip
Make sure StoredPlaylists always contain UTF-8 paths that are either
relative paths in the DB or URLs. The main functional difference is that playlistmove and playlistdelete will rewrite playlists in the correct encoding and remove invalid lines instead of potentially modifying them. The specific changes are: appendSongToStoredPlaylist: * Don't convert to FS charset * Don't prepend music_directory if saving absolute paths writeStoredPlaylistToPath: * Convert to FS charset * Prepend music_directory if saving absolute paths loadStoredPlaylist: * Make sure each line is either in the DB or a URL loadPlaylist: * Don't bother checking paths, since it's done in loadStoredPlaylist now git-svn-id: https://svn.musicpd.org/mpd/trunk@6266 09075e82-0dd4-0310-85a5-a0d7c8717e4f
Diffstat (limited to '')
-rw-r--r--src/playlist.c4
-rw-r--r--src/storedPlaylist.c57
2 files changed, 38 insertions, 23 deletions
diff --git a/src/playlist.c b/src/playlist.c
index d389b1db8..5536dcf8d 100644
--- a/src/playlist.c
+++ b/src/playlist.c
@@ -1434,9 +1434,7 @@ int loadPlaylist(int fd, char *utf8file)
node = sp->list->firstNode;
while (node != NULL) {
char *temp = node->data;
- if (!getSongFromDB(temp) && !isRemoteUrl(temp)) {
-
- } else if ((addToPlaylist(STDERR_FILENO, temp, 0)) < 0) {
+ if ((addToPlaylist(STDERR_FILENO, temp, 0)) < 0) {
/* for windows compatibility, convert slashes */
char *temp2 = xstrdup(temp);
char *p = temp2;
diff --git a/src/storedPlaylist.c b/src/storedPlaylist.c
index a4b135716..9cdfeabf0 100644
--- a/src/storedPlaylist.c
+++ b/src/storedPlaylist.c
@@ -23,6 +23,8 @@
#include "playlist.h"
#include "ack.h"
#include "command.h"
+#include "ls.h"
+#include "directory.h"
#include <string.h>
#include <errno.h>
@@ -96,6 +98,11 @@ static ListNode *nodeOfStoredPlaylist(StoredPlaylist *sp, int index)
return NULL;
}
+static void appendSongToStoredPlaylist(StoredPlaylist *sp, Song *song)
+{
+ insertInListWithoutKey(sp->list, strdup(getSongUrl(song)));
+}
+
StoredPlaylist *newStoredPlaylist(const char *utf8name, int fd, int ignoreExisting)
{
struct stat buf;
@@ -138,6 +145,7 @@ StoredPlaylist *loadStoredPlaylist(const char *utf8path, int fd)
int parentlen = strlen(parent);
int tempInt;
int commentCharFound = 0;
+ Song *song;
filename = utf8pathToFsPathInStoredPlaylist(utf8path, fd);
if (!filename)
@@ -182,10 +190,22 @@ StoredPlaylist *loadStoredPlaylist(const char *utf8path, int fd)
}
slength = 0;
temp = fsCharsetToUtf8(s);
- if (!temp)
- continue;
- if (!commentCharFound)
- insertInListWithoutKey(sp->list, strdup(s));
+ if (temp && !commentCharFound) {
+ song = getSongFromDB(temp);
+ if (song) {
+ appendSongToStoredPlaylist(sp, song);
+ continue;
+ }
+
+ if (!isValidRemoteUtf8Url(temp))
+ continue;
+
+ song = newSong(temp, SONG_TYPE_URL, NULL);
+ if (song) {
+ appendSongToStoredPlaylist(sp, song);
+ freeJustSong(song);
+ }
+ }
} else if (slength == MAXPATHLEN) {
s[slength] = '\0';
commandError(sp->fd, ACK_ERROR_PLAYLIST_LOAD,
@@ -365,8 +385,9 @@ int removeOneSongFromStoredPlaylistByPath(int fd, const char *utf8path, int pos)
static int writeStoredPlaylistToPath(StoredPlaylist *sp, const char *fspath)
{
- FILE *file;
ListNode *node;
+ FILE *file;
+ char *s;
if (fspath == NULL)
return -1;
@@ -380,7 +401,12 @@ static int writeStoredPlaylistToPath(StoredPlaylist *sp, const char *fspath)
node = sp->list->firstNode;
while (node != NULL) {
- fprintf(file, "%s\n", (char *)node->data);
+ s = (char *)node->data;
+ if (isValidRemoteUtf8Url(s) || !playlist_saveAbsolutePaths)
+ s = utf8ToFsCharset(s);
+ else
+ s = rmp2amp(utf8ToFsCharset(s));
+ fprintf(file, "%s\n", s);
node = node->nextNode;
}
@@ -393,22 +419,11 @@ int writeStoredPlaylist(StoredPlaylist *sp)
return writeStoredPlaylistToPath(sp, sp->fspath);
}
-static void appendSongToStoredPlaylist(StoredPlaylist *sp, Song *song)
-{
- char *s;
-
- if (playlist_saveAbsolutePaths && song->type == SONG_TYPE_FILE)
- s = rmp2amp(utf8ToFsCharset(getSongUrl(song)));
- else
- s = utf8ToFsCharset(getSongUrl(song));
-
- insertInListWithoutKey(sp->list, strdup(s));
-}
-
int appendSongToStoredPlaylistByPath(int fd, const char *utf8path, Song *song)
{
char *filename;
FILE *file;
+ char *s;
filename = utf8pathToFsPathInStoredPlaylist(utf8path, fd);
if (!filename)
@@ -422,9 +437,11 @@ int appendSongToStoredPlaylistByPath(int fd, const char *utf8path, Song *song)
}
if (playlist_saveAbsolutePaths && song->type == SONG_TYPE_FILE)
- fprintf(file, "%s\n", rmp2amp(utf8ToFsCharset(getSongUrl(song))));
+ s = rmp2amp(utf8ToFsCharset(getSongUrl(song)));
else
- fprintf(file, "%s\n", utf8ToFsCharset(getSongUrl(song)));
+ s = utf8ToFsCharset(getSongUrl(song));
+
+ fprintf(file, "%s\n", s);
while (fclose(file) != 0 && errno == EINTR);
return 0;