aboutsummaryrefslogtreecommitdiffstats
path: root/src/update_walk.c
diff options
context:
space:
mode:
authorMax Kellermann <max@duempel.org>2012-02-12 17:06:34 +0100
committerMax Kellermann <max@duempel.org>2012-02-12 17:41:34 +0100
commit4fdcc0496fbfce7b1ad4f46b249e4c9ee0b0e997 (patch)
treea24e9f1a0f65551359e1fa607901d51d4fb6cdf1 /src/update_walk.c
parent0c4a2bea6917cacd562e61f50727dcc68944f3ad (diff)
downloadmpd-4fdcc0496fbfce7b1ad4f46b249e4c9ee0b0e997.tar.gz
mpd-4fdcc0496fbfce7b1ad4f46b249e4c9ee0b0e997.tar.xz
mpd-4fdcc0496fbfce7b1ad4f46b249e4c9ee0b0e997.zip
update_walk: move code to update_db.c
Diffstat (limited to 'src/update_walk.c')
-rw-r--r--src/update_walk.c89
1 files changed, 4 insertions, 85 deletions
diff --git a/src/update_walk.c b/src/update_walk.c
index cc0749e6d..e2f7d3315 100644
--- a/src/update_walk.c
+++ b/src/update_walk.c
@@ -20,6 +20,7 @@
#include "config.h" /* must be first for large file support */
#include "update_internal.h"
#include "update_io.h"
+#include "update_db.h"
#include "database.h"
#include "db_lock.h"
#include "exclude.h"
@@ -93,88 +94,6 @@ directory_set_stat(struct directory *dir, const struct stat *st)
dir->have_stat = true;
}
-/**
- * Caller must lock the #db_mutex.
- */
-static void
-delete_song(struct directory *dir, struct song *del)
-{
- assert(del->parent == dir);
-
- /* first, prevent traversers in main task from getting this */
- directory_remove_song(dir, del);
-
- db_unlock(); /* temporary unlock, because update_remove_song() blocks */
-
- /* now take it out of the playlist (in the main_task) */
- update_remove_song(del);
-
- /* finally, all possible references gone, free it */
- song_free(del);
-
- db_lock();
-}
-
-static void
-delete_directory(struct directory *directory);
-
-/**
- * Recursively remove all sub directories and songs from a directory,
- * leaving an empty directory.
- *
- * Caller must lock the #db_mutex.
- */
-static void
-clear_directory(struct directory *directory)
-{
- struct directory *child, *n;
- directory_for_each_child_safe(child, n, directory)
- delete_directory(child);
-
- struct song *song, *ns;
- directory_for_each_song_safe(song, ns, directory) {
- assert(song->parent == directory);
- delete_song(directory, song);
- }
-}
-
-/**
- * Recursively free a directory and all its contents.
- *
- * Caller must lock the #db_mutex.
- */
-static void
-delete_directory(struct directory *directory)
-{
- assert(directory->parent != NULL);
-
- clear_directory(directory);
-
- directory_delete(directory);
-}
-
-static void
-delete_name_in(struct directory *parent, const char *name)
-{
- db_lock();
- struct directory *directory = directory_get_child(parent, name);
-
- if (directory != NULL) {
- delete_directory(directory);
- modified = true;
- }
-
- struct song *song = directory_get_song(parent, name);
- if (song != NULL) {
- delete_song(parent, song);
- modified = true;
- }
-
- db_unlock();
-
- playlist_vector_remove(&parent->playlists, name);
-}
-
static void
remove_excluded_from_directory(struct directory *directory,
GSList *exclude_list)
@@ -717,7 +636,7 @@ updateDirectory(struct directory *directory, const struct stat *st)
continue;
if (skip_symlink(directory, utf8)) {
- delete_name_in(directory, utf8);
+ modified |= delete_name_in(directory, utf8);
g_free(utf8);
continue;
}
@@ -725,7 +644,7 @@ updateDirectory(struct directory *directory, const struct stat *st)
if (stat_directory_child(directory, utf8, &st2) == 0)
updateInDirectory(directory, utf8, &st2);
else
- delete_name_in(directory, utf8);
+ modified |= delete_name_in(directory, utf8);
g_free(utf8);
}
@@ -810,7 +729,7 @@ updatePath(const char *path)
if (stat_directory_child(parent, name, &st) == 0)
updateInDirectory(parent, name, &st);
else
- delete_name_in(parent, name);
+ modified |= delete_name_in(parent, name);
g_free(name);
}