aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMax Kellermann <max@duempel.org>2013-01-30 18:54:17 +0100
committerMax Kellermann <max@duempel.org>2013-01-30 21:36:11 +0100
commit72070f292b98ee8b38217b43dc046b047bcccd8e (patch)
tree929a3e1b2a66a343d3b8ed9194d104a12bb2de91
parent823c618d241e7e45f31981c437215bc35664c385 (diff)
ConfigFile: add struct ConfigData
-rw-r--r--src/ConfigData.hxx18
-rw-r--r--src/ConfigFile.cxx17
2 files changed, 26 insertions, 9 deletions
diff --git a/src/ConfigData.hxx b/src/ConfigData.hxx
index d156cef1..98b74635 100644
--- a/src/ConfigData.hxx
+++ b/src/ConfigData.hxx
@@ -20,11 +20,19 @@
#ifndef MPD_CONFIG_DATA_HXX
#define MPD_CONFIG_DATA_HXX
+#include "ConfigOption.hxx"
#include "gerror.h"
#include "gcc.h"
+#ifdef __cplusplus
+#include <glib.h>
+#include <array>
+#endif
+
#include <stdbool.h>
+#ifdef __cplusplus
+
struct block_param {
char *name;
char *value;
@@ -37,6 +45,8 @@ struct block_param {
bool used;
};
+#endif
+
struct config_param {
char *value;
unsigned int line;
@@ -52,6 +62,14 @@ struct config_param {
};
#ifdef __cplusplus
+
+struct ConfigData {
+ std::array<GSList *, std::size_t(CONF_MAX)> params;
+};
+
+#endif
+
+#ifdef __cplusplus
extern "C" {
#endif
diff --git a/src/ConfigFile.cxx b/src/ConfigFile.cxx
index af619c0b..d10c9046 100644
--- a/src/ConfigFile.cxx
+++ b/src/ConfigFile.cxx
@@ -47,7 +47,7 @@ extern "C" {
#define CONF_COMMENT '#'
-static GSList *config_params[unsigned(CONF_MAX)];
+static ConfigData config_data;
static void
config_param_free_callback(gpointer data, G_GNUC_UNUSED gpointer user_data)
@@ -59,10 +59,9 @@ config_param_free_callback(gpointer data, G_GNUC_UNUSED gpointer user_data)
void config_global_finish(void)
{
- for (unsigned i = 0; i < G_N_ELEMENTS(config_params); ++i) {
- g_slist_foreach(config_params[i],
- config_param_free_callback, NULL);
- g_slist_free(config_params[i]);
+ for (auto i : config_data.params) {
+ g_slist_foreach(i, config_param_free_callback, NULL);
+ g_slist_free(i);
}
}
@@ -92,8 +91,8 @@ config_param_check(gpointer data, G_GNUC_UNUSED gpointer user_data)
void config_global_check(void)
{
- for (unsigned i = 0; i < G_N_ELEMENTS(config_params); ++i)
- g_slist_foreach(config_params[i], config_param_check, NULL);
+ for (auto i : config_data.params)
+ g_slist_foreach(i, config_param_check, NULL);
}
static bool
@@ -244,7 +243,7 @@ ReadConfigFile(const Path &path, GError **error_r)
const unsigned i = ParseConfigOptionName(name);
const ConfigTemplate &option = config_templates[i];
- GSList *&params = config_params[i];
+ GSList *&params = config_data.params[i];
if (params != NULL && !option.repeatable) {
param = (struct config_param *)params->data;
@@ -323,7 +322,7 @@ ReadConfigFile(const Path &path, GError **error_r)
const struct config_param *
config_get_next_param(ConfigOption option, const struct config_param * last)
{
- GSList *node = config_params[unsigned(option)];
+ GSList *node = config_data.params[unsigned(option)];
if (last) {
node = g_slist_find(node, last);