From 04618898d641a473c072e947fd7774146a25cbd6 Mon Sep 17 00:00:00 2001 From: Max Kellermann Date: Wed, 13 Jun 2012 22:14:16 +0200 Subject: update_walk: move code to update_song.c --- src/update_song.c | 114 ++++++++++++++++++++++++++++++++++++++++++++++++++++++ src/update_song.h | 35 +++++++++++++++++ src/update_walk.c | 86 +--------------------------------------- 3 files changed, 151 insertions(+), 84 deletions(-) create mode 100644 src/update_song.c create mode 100644 src/update_song.h (limited to 'src') diff --git a/src/update_song.c b/src/update_song.c new file mode 100644 index 00000000..5f06985a --- /dev/null +++ b/src/update_song.c @@ -0,0 +1,114 @@ +/* + * Copyright (C) 2003-2012 The Music Player Daemon Project + * http://www.musicpd.org + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation; either version 2 of the License, or + * (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * You should have received a copy of the GNU General Public License along + * with this program; if not, write to the Free Software Foundation, Inc., + * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. + */ + +#include "config.h" /* must be first for large file support */ +#include "update_song.h" +#include "update_internal.h" +#include "update_io.h" +#include "update_db.h" +#include "update_container.h" +#include "db_lock.h" +#include "directory.h" +#include "song.h" +#include "glib_compat.h" +#include "decoder_list.h" +#include "decoder_plugin.h" + +#include + +#include + +static void +update_song_file2(struct directory *directory, + const char *name, const struct stat *st, + const struct decoder_plugin *plugin) +{ + db_lock(); + struct song *song = directory_get_song(directory, name); + db_unlock(); + + if (!directory_child_access(directory, name, R_OK)) { + g_warning("no read permissions on %s/%s", + directory_get_path(directory), name); + if (song != NULL) { + db_lock(); + delete_song(directory, song); + db_unlock(); + } + + return; + } + + if (!(song != NULL && st->st_mtime == song->mtime && + !walk_discard) && + update_container_file(directory, name, st, plugin)) { + if (song != NULL) { + db_lock(); + delete_song(directory, song); + db_unlock(); + } + + return; + } + + if (song == NULL) { + g_debug("reading %s/%s", + directory_get_path(directory), name); + song = song_file_load(name, directory); + if (song == NULL) { + g_debug("ignoring unrecognized file %s/%s", + directory_get_path(directory), name); + return; + } + + db_lock(); + directory_add_song(directory, song); + db_unlock(); + + modified = true; + g_message("added %s/%s", + directory_get_path(directory), name); + } else if (st->st_mtime != song->mtime || walk_discard) { + g_message("updating %s/%s", + directory_get_path(directory), name); + if (!song_file_update(song)) { + g_debug("deleting unrecognized file %s/%s", + directory_get_path(directory), name); + db_lock(); + delete_song(directory, song); + db_unlock(); + } + + modified = true; + } +} + +bool +update_song_file(struct directory *directory, + const char *name, const char *suffix, + const struct stat *st) +{ + const struct decoder_plugin *plugin = + decoder_plugin_from_suffix(suffix, false); + if (plugin == NULL) + return false; + + update_song_file2(directory, name, st, plugin); + return true; +} diff --git a/src/update_song.h b/src/update_song.h new file mode 100644 index 00000000..cff63f57 --- /dev/null +++ b/src/update_song.h @@ -0,0 +1,35 @@ +/* + * Copyright (C) 2003-2012 The Music Player Daemon Project + * http://www.musicpd.org + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation; either version 2 of the License, or + * (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * You should have received a copy of the GNU General Public License along + * with this program; if not, write to the Free Software Foundation, Inc., + * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. + */ + +#ifndef MPD_UPDATE_SONG_H +#define MPD_UPDATE_SONG_H + +#include "check.h" + +#include +#include + +struct directory; + +bool +update_song_file(struct directory *directory, + const char *name, const char *suffix, + const struct stat *st); + +#endif diff --git a/src/update_walk.c b/src/update_walk.c index 061a446f..23c5d3e4 100644 --- a/src/update_walk.c +++ b/src/update_walk.c @@ -19,10 +19,9 @@ #include "config.h" /* must be first for large file support */ #include "update_walk.h" -#include "update_internal.h" #include "update_io.h" #include "update_db.h" -#include "update_container.h" +#include "update_song.h" #include "update_archive.h" #include "database.h" #include "db_lock.h" @@ -33,8 +32,6 @@ #include "uri.h" #include "mapper.h" #include "path.h" -#include "decoder_list.h" -#include "decoder_plugin.h" #include "playlist_list.h" #include "glib_compat.h" #include "conf.h" @@ -207,85 +204,6 @@ find_inode_ancestor(struct directory *parent, ino_t inode, dev_t device) return 0; } -static void -update_song_file(struct directory *directory, - const char *name, const struct stat *st, - const struct decoder_plugin *plugin) -{ - db_lock(); - struct song *song = directory_get_song(directory, name); - db_unlock(); - - if (!directory_child_access(directory, name, R_OK)) { - g_warning("no read permissions on %s/%s", - directory_get_path(directory), name); - if (song != NULL) { - db_lock(); - delete_song(directory, song); - db_unlock(); - } - - return; - } - - if (!(song != NULL && st->st_mtime == song->mtime && - !walk_discard) && - update_container_file(directory, name, st, plugin)) { - if (song != NULL) { - db_lock(); - delete_song(directory, song); - db_unlock(); - } - - return; - } - - if (song == NULL) { - g_debug("reading %s/%s", - directory_get_path(directory), name); - song = song_file_load(name, directory); - if (song == NULL) { - g_debug("ignoring unrecognized file %s/%s", - directory_get_path(directory), name); - return; - } - - db_lock(); - directory_add_song(directory, song); - db_unlock(); - - modified = true; - g_message("added %s/%s", - directory_get_path(directory), name); - } else if (st->st_mtime != song->mtime || walk_discard) { - g_message("updating %s/%s", - directory_get_path(directory), name); - if (!song_file_update(song)) { - g_debug("deleting unrecognized file %s/%s", - directory_get_path(directory), name); - db_lock(); - delete_song(directory, song); - db_unlock(); - } - - modified = true; - } -} - -static bool -update_song_file2(struct directory *directory, - const char *name, const char *suffix, - const struct stat *st) -{ - const struct decoder_plugin *plugin = - decoder_plugin_from_suffix(suffix, false); - if (plugin == NULL) - return false; - - update_song_file(directory, name, st, plugin); - return true; -} - static bool update_playlist_file2(struct directory *directory, const char *name, const char *suffix, @@ -310,7 +228,7 @@ update_regular_file(struct directory *directory, if (suffix == NULL) return false; - return update_song_file2(directory, name, suffix, st) || + return update_song_file(directory, name, suffix, st) || update_archive_file(directory, name, suffix, st) || update_playlist_file2(directory, name, suffix, st); } -- cgit v1.2.3