aboutsummaryrefslogtreecommitdiffstats
path: root/src
diff options
context:
space:
mode:
authorMax Kellermann <max@duempel.org>2009-01-18 16:09:01 +0100
committerMax Kellermann <max@duempel.org>2009-01-18 16:09:01 +0100
commitc2cc3b4923684cb00518e7b8db25a9b56c60dc9d (patch)
tree6f90b4fd00c6a07e90b1f3b555fb9819faebc2de /src
parent004dfddca383394426f6d191a43c9a0131cdee34 (diff)
downloadmpd-c2cc3b4923684cb00518e7b8db25a9b56c60dc9d.tar.gz
mpd-c2cc3b4923684cb00518e7b8db25a9b56c60dc9d.tar.xz
mpd-c2cc3b4923684cb00518e7b8db25a9b56c60dc9d.zip
database: pass database file name to db_init()
Don't include conf.h in database.c.
Diffstat (limited to 'src')
-rw-r--r--src/database.c68
-rw-r--r--src/database.h12
-rw-r--r--src/main.c5
3 files changed, 45 insertions, 40 deletions
diff --git a/src/database.c b/src/database.c
index cb9d92819..44909aced 100644
--- a/src/database.c
+++ b/src/database.c
@@ -21,7 +21,6 @@
#include "directory.h"
#include "directory_save.h"
#include "song.h"
-#include "conf.h"
#include "path.h"
#include "stats.h"
#include "config.h"
@@ -39,13 +38,17 @@
#undef G_LOG_DOMAIN
#define G_LOG_DOMAIN "database"
+static char *database_path;
+
static struct directory *music_root;
static time_t directory_dbModTime;
void
-db_init(void)
+db_init(const char *path)
{
+ database_path = g_strdup(path);
+
music_root = directory_new("", NULL);
}
@@ -53,6 +56,15 @@ void
db_finish(void)
{
directory_free(music_root);
+
+ g_free(database_path);
+}
+
+void
+db_clear(void)
+{
+ directory_free(music_root);
+ music_root = directory_new("", NULL);
}
struct directory *
@@ -120,42 +132,32 @@ db_walk(const char *name,
return directory_walk(directory, forEachSong, forEachDir, data);
}
-static char *
-db_get_file(void)
-{
- struct config_param *param = parseConfigFilePath(CONF_DB_FILE, 1);
-
- assert(param);
- assert(param->value);
-
- return param->value;
-}
-
bool
db_check(void)
{
struct stat st;
- char *dbFile = db_get_file();
+
+ assert(database_path != NULL);
/* Check if the file exists */
- if (access(dbFile, F_OK)) {
+ if (access(database_path, F_OK)) {
/* If the file doesn't exist, we can't check if we can write
* it, so we are going to try to get the directory path, and
* see if we can write a file in that */
- char *dirPath = g_path_get_dirname(dbFile);
+ char *dirPath = g_path_get_dirname(database_path);
/* Check that the parent part of the path is a directory */
if (stat(dirPath, &st) < 0) {
g_free(dirPath);
g_warning("Couldn't stat parent directory of db file "
- "\"%s\": %s", dbFile, strerror(errno));
+ "\"%s\": %s", database_path, strerror(errno));
return false;
}
if (!S_ISDIR(st.st_mode)) {
g_free(dirPath);
g_warning("Couldn't create db file \"%s\" because the "
- "parent path is not a directory", dbFile);
+ "parent path is not a directory", database_path);
return false;
}
@@ -173,21 +175,21 @@ db_check(void)
}
/* Path exists, now check if it's a regular file */
- if (stat(dbFile, &st) < 0) {
+ if (stat(database_path, &st) < 0) {
g_warning("Couldn't stat db file \"%s\": %s",
- dbFile, strerror(errno));
+ database_path, strerror(errno));
return false;
}
if (!S_ISREG(st.st_mode)) {
- g_warning("db file \"%s\" is not a regular file", dbFile);
+ g_warning("db file \"%s\" is not a regular file", database_path);
return false;
}
/* And check that we can write to it */
- if (access(dbFile, R_OK | W_OK)) {
+ if (access(database_path, R_OK | W_OK)) {
g_warning("Can't open db file \"%s\" for reading/writing: %s",
- dbFile, strerror(errno));
+ database_path, strerror(errno));
return false;
}
@@ -198,9 +200,11 @@ bool
db_save(void)
{
FILE *fp;
- char *dbFile = db_get_file();
struct stat st;
+ assert(database_path != NULL);
+ assert(music_root != NULL);
+
g_debug("removing empty directories from DB");
directory_prune_empty(music_root);
@@ -210,10 +214,10 @@ db_save(void)
g_debug("writing DB");
- fp = fopen(dbFile, "w");
+ fp = fopen(database_path, "w");
if (!fp) {
g_warning("unable to write to db file \"%s\": %s",
- dbFile, strerror(errno));
+ database_path, strerror(errno));
return false;
}
@@ -232,7 +236,7 @@ db_save(void)
while (fclose(fp) && errno == EINTR);
- if (stat(dbFile, &st) == 0)
+ if (stat(database_path, &st) == 0)
directory_dbModTime = st.st_mtime;
return true;
@@ -242,19 +246,19 @@ bool
db_load(void)
{
FILE *fp = NULL;
- char *dbFile = db_get_file();
struct stat st;
char buffer[100];
bool foundFsCharset = false, foundVersion = false;
+ assert(database_path != NULL);
assert(music_root != NULL);
if (!music_root)
music_root = directory_new("", NULL);
- while (!(fp = fopen(dbFile, "r")) && errno == EINTR) ;
+ while (!(fp = fopen(database_path, "r")) && errno == EINTR) ;
if (fp == NULL) {
g_warning("unable to open db file \"%s\": %s",
- dbFile, strerror(errno));
+ database_path, strerror(errno));
return false;
}
@@ -289,7 +293,7 @@ db_load(void)
foundFsCharset = true;
fsCharset = &(buffer[strlen(DIRECTORY_FS_CHARSET)]);
- tempCharset = config_get_string(CONF_FS_CHARSET, NULL);
+ tempCharset = path_get_fs_charset();
if (tempCharset != NULL
&& strcmp(fsCharset, tempCharset)) {
g_message("Using \"%s\" for the "
@@ -312,7 +316,7 @@ db_load(void)
stats_update();
- if (stat(dbFile, &st) == 0)
+ if (stat(database_path, &st) == 0)
directory_dbModTime = st.st_mtime;
return true;
diff --git a/src/database.h b/src/database.h
index f0f225108..bb2ccce70 100644
--- a/src/database.h
+++ b/src/database.h
@@ -27,9 +27,11 @@ struct directory;
/**
* Initialize the database library.
+ *
+ * @param path the absolute path of the database file
*/
void
-db_init(void);
+db_init(const char *path);
void
db_finish(void);
@@ -37,12 +39,8 @@ db_finish(void);
/**
* Clear the database.
*/
-static inline void
-db_clear(void)
-{
- db_finish();
- db_init();
-}
+void
+db_clear(void);
struct directory *
db_get_root(void);
diff --git a/src/main.c b/src/main.c
index ee6efd194..b7e9e0106 100644
--- a/src/main.c
+++ b/src/main.c
@@ -126,7 +126,10 @@ static void changeToUser(void)
static void openDB(Options * options, char *argv0)
{
- db_init();
+ struct config_param *param;
+
+ param = parseConfigFilePath(CONF_DB_FILE, true);
+ db_init(param->value);
if (options->createDB > 0 || !db_load()) {
unsigned job;