aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMax Kellermann <max@duempel.org>2009-01-18 18:59:10 +0100
committerMax Kellermann <max@duempel.org>2009-01-18 18:59:10 +0100
commit70c6cc33f04693c6a2a574541cce076fa84c86a5 (patch)
tree1e21b5c606e3a29f5fbfcb972ad2f14e964e55c8
parenta0603d8897a7066564172c086b6e532d8a5a40c5 (diff)
downloadmpd-70c6cc33f04693c6a2a574541cce076fa84c86a5.tar.gz
mpd-70c6cc33f04693c6a2a574541cce076fa84c86a5.tar.xz
mpd-70c6cc33f04693c6a2a574541cce076fa84c86a5.zip
conf: removed parseConfigFilePath()
Use config_get_path() instead in mapper.c.
-rw-r--r--src/conf.c23
-rw-r--r--src/conf.h3
-rw-r--r--src/mapper.c39
3 files changed, 19 insertions, 46 deletions
diff --git a/src/conf.c b/src/conf.c
index adc8ad876..a073f32a5 100644
--- a/src/conf.c
+++ b/src/conf.c
@@ -425,29 +425,6 @@ getBlockParam(struct config_param * param, const char *name)
return ret;
}
-struct config_param *
-parseConfigFilePath(const char *name, int force)
-{
- struct config_param *param = config_get_param(name);
- char *path;
-
- if (!param && force)
- g_error("config parameter \"%s\" not found\n", name);
-
- if (!param)
- return NULL;
-
- path = parsePath(param->value);
- if (!path)
- g_error("error parsing \"%s\" at line %i\n",
- name, param->line);
-
- g_free(param->value);
- param->value = path;
-
- return param;
-}
-
bool config_get_bool(const char *name, bool default_value)
{
struct config_param *param = config_get_param(name);
diff --git a/src/conf.h b/src/conf.h
index 283f763a2..dda328ba9 100644
--- a/src/conf.h
+++ b/src/conf.h
@@ -112,9 +112,6 @@ config_get_path(const char *name);
struct block_param *
getBlockParam(struct config_param *param, const char *name);
-struct config_param *
-parseConfigFilePath(const char *name, int force);
-
bool config_get_bool(const char *name, bool default_value);
bool
diff --git a/src/mapper.c b/src/mapper.c
index a47507bb1..8de2032d9 100644
--- a/src/mapper.c
+++ b/src/mapper.c
@@ -41,7 +41,7 @@ static size_t music_dir_length;
static char *playlist_dir;
static void
-mapper_set_music_dir(const char *path, int line)
+mapper_set_music_dir(const char *path)
{
int ret;
struct stat st;
@@ -51,15 +51,15 @@ mapper_set_music_dir(const char *path, int line)
ret = stat(music_dir, &st);
if (ret < 0)
- g_warning("failed to stat music directory \"%s\" (config line %i): %s\n",
- music_dir, line, g_strerror(errno));
+ g_warning("failed to stat music directory \"%s\": %s",
+ music_dir, g_strerror(errno));
else if (!S_ISDIR(st.st_mode))
- g_warning("music directory is not a directory: \"%s\" (config line %i)\n",
- music_dir, line);
+ g_warning("music directory is not a directory: \"%s\"",
+ music_dir);
}
static void
-mapper_set_playlist_dir(const char *path, int line)
+mapper_set_playlist_dir(const char *path)
{
int ret;
struct stat st;
@@ -68,32 +68,31 @@ mapper_set_playlist_dir(const char *path, int line)
ret = stat(playlist_dir, &st);
if (ret < 0)
- g_warning("failed to stat playlist directory \"%s\" (config line %i): %s\n",
- playlist_dir, line, g_strerror(errno));
+ g_warning("failed to stat playlist directory \"%s\": %s",
+ playlist_dir, g_strerror(errno));
else if (!S_ISDIR(st.st_mode))
- g_warning("playlist directory is not a directory: \"%s\" (config line %i)\n",
- playlist_dir, line);
+ g_warning("playlist directory is not a directory: \"%s\"",
+ playlist_dir);
}
void mapper_init(void)
{
- struct config_param *param;
+ const char *path;
- param = parseConfigFilePath(CONF_MUSIC_DIR, false);
- if (param != NULL)
- mapper_set_music_dir(param->value, param->line);
+ path = config_get_path(CONF_MUSIC_DIR);
+ if (path != NULL)
+ mapper_set_music_dir(path);
#if GLIB_MAJOR_VERSION > 2 || (GLIB_MAJOR_VERSION == 2 && GLIB_MINOR_VERSION >= 14)
else {
- const char *path =
- g_get_user_special_dir(G_USER_DIRECTORY_MUSIC);
+ path = g_get_user_special_dir(G_USER_DIRECTORY_MUSIC);
if (path != NULL)
- mapper_set_music_dir(path, -1);
+ mapper_set_music_dir(path);
}
#endif
- param = parseConfigFilePath(CONF_PLAYLIST_DIR, false);
- if (param != NULL)
- mapper_set_playlist_dir(param->value, param->line);
+ path = config_get_path(CONF_PLAYLIST_DIR);
+ if (path != NULL)
+ mapper_set_playlist_dir(path);
}
void mapper_finish(void)