aboutsummaryrefslogtreecommitdiffstats
path: root/src
diff options
context:
space:
mode:
authorMax Kellermann <max@duempel.org>2008-10-09 19:13:02 +0200
committerEric Wong <normalperson@yhbt.net>2008-10-11 19:21:50 -0700
commit2d6a943bf0c84a2ece6014d7a74b25f819bc4510 (patch)
treeaff5ae339c512a18b21899a3fc7a579d323acc5f /src
parent6fd08bc8fad5d6c4be37ce751d53ef80b756b292 (diff)
downloadmpd-2d6a943bf0c84a2ece6014d7a74b25f819bc4510.tar.gz
mpd-2d6a943bf0c84a2ece6014d7a74b25f819bc4510.tar.xz
mpd-2d6a943bf0c84a2ece6014d7a74b25f819bc4510.zip
update: rewrote updatePath() using updateInDirectory()
updatePath() duplicated a lot of code from the more generic updateInDirectory(). Eliminate most of updatePath() and call updateInDirectory().
Diffstat (limited to 'src')
-rw-r--r--src/update.c89
1 files changed, 19 insertions, 70 deletions
diff --git a/src/update.c b/src/update.c
index 641b1d63b..e1d0268a8 100644
--- a/src/update.c
+++ b/src/update.c
@@ -140,6 +140,21 @@ static int delete_song_if_removed(struct mpd_song *song, void *_data)
return 0;
}
+static enum update_return delete_path(const char *path)
+{
+ struct directory *directory = db_get_directory(path);
+ struct mpd_song *song = db_get_song(path);
+
+ if (directory)
+ delete_directory(directory);
+ if (song)
+ delete_song(song->parent, song);
+
+ return directory == NULL && song == NULL
+ ? UPDATE_RETURN_NOUPDATE
+ : UPDATE_RETURN_UPDATED;
+}
+
static enum update_return
removeDeletedFromDirectory(char *path_max_tmp, struct directory *directory)
{
@@ -368,77 +383,11 @@ addParentPathToDB(const char *utf8path)
static enum update_return updatePath(const char *utf8path)
{
- struct directory *directory;
- struct directory *parentDirectory;
- struct mpd_song *song;
- time_t mtime;
- enum update_return ret = UPDATE_RETURN_NOUPDATE;
- char path_max_tmp[MPD_PATH_MAX];
-
- assert(utf8path);
-
- /* if path is in the DB try to update it, or else delete it */
- if ((directory = db_get_directory(utf8path))) {
- parentDirectory = directory->parent;
-
- /* if this update directory is successfull, we are done */
- if ((ret = updateDirectory(directory)) >= 0) {
- directory_sort(directory);
- return ret;
- }
- /* if updateDirectory fails, means we should delete it */
- else {
- LOG("removing directory: %s\n", utf8path);
- delete_directory(directory);
- ret = UPDATE_RETURN_UPDATED;
- /* don't return, path maybe a song now */
- }
- } else if ((song = db_get_song(utf8path))) {
- parentDirectory = song->parent;
- if (!parentDirectory->stat
- && statDirectory(parentDirectory) < 0) {
- return UPDATE_RETURN_NOUPDATE;
- }
- /* if this song update is successful, we are done */
- else if (!inodeFoundInParent(parentDirectory->parent,
- parentDirectory->inode,
- parentDirectory->device) &&
- isMusic(song_get_url(song, path_max_tmp), &mtime, 0)) {
- if (song->mtime == mtime)
- return UPDATE_RETURN_NOUPDATE;
- else if (song_file_update(song))
- return UPDATE_RETURN_UPDATED;
- else {
- delete_song(parentDirectory, song);
- return UPDATE_RETURN_UPDATED;
- }
- }
- /* if updateDirectory fails, means we should delete it */
- else {
- delete_song(parentDirectory, song);
- ret = UPDATE_RETURN_UPDATED;
- /* don't return, path maybe a directory now */
- }
- }
-
- /* path not found in the db, see if it actually exists on the fs.
- * Also, if by chance a directory was replaced by a file of the same
- * name or vice versa, we need to add it to the db
- */
- if (isDir(utf8path) || isMusic(utf8path, NULL, 0)) {
- parentDirectory = addParentPathToDB(utf8path);
- if (!parentDirectory || (!parentDirectory->stat &&
- statDirectory(parentDirectory) < 0)) {
- } else if (0 == inodeFoundInParent(parentDirectory->parent,
- parentDirectory->inode,
- parentDirectory->device)
- && updateInDirectory(parentDirectory, utf8path)
- == UPDATE_RETURN_UPDATED) {
- ret = UPDATE_RETURN_UPDATED;
- }
- }
+ struct stat st;
- return ret;
+ if (myStat(path, &st) < 0)
+ return delete_path(path);
+ return updateInDirectory(addParentPathToDB(path), path);
}
static void * update_task(void *_path)