aboutsummaryrefslogtreecommitdiffstats
path: root/src/input_init.c
diff options
context:
space:
mode:
authorMax Kellermann <max@duempel.org>2009-12-14 22:29:46 +0100
committerMax Kellermann <max@duempel.org>2009-12-14 23:16:18 +0100
commit786c1f035f99f3a396dfd52a469601163de3e9af (patch)
tree152f9a1e628d1eb8bdb0f67fa6348b88d98cfb7d /src/input_init.c
parentf70d2f58a1ac221d488a4fbb2fda39d83048ec53 (diff)
downloadmpd-786c1f035f99f3a396dfd52a469601163de3e9af.tar.gz
mpd-786c1f035f99f3a396dfd52a469601163de3e9af.tar.xz
mpd-786c1f035f99f3a396dfd52a469601163de3e9af.zip
input_plugin: method init() returns errors with GError
Not used by any plugin currently, but this eliminates the g_error() call in input_plugin_config(), so it's worth it.
Diffstat (limited to '')
-rw-r--r--src/input_init.c39
1 files changed, 32 insertions, 7 deletions
diff --git a/src/input_init.c b/src/input_init.c
index 5f45de5c7..c4d015594 100644
--- a/src/input_init.c
+++ b/src/input_init.c
@@ -22,9 +22,16 @@
#include "input_plugin.h"
#include "input_registry.h"
#include "conf.h"
+#include "glib_compat.h"
#include <string.h>
+static inline GQuark
+input_quark(void)
+{
+ return g_quark_from_static_string("input");
+}
+
/**
* Find the "input" configuration block for the specified plugin.
*
@@ -32,16 +39,19 @@
* @return the configuration block, or NULL if none was configured
*/
static const struct config_param *
-input_plugin_config(const char *plugin_name)
+input_plugin_config(const char *plugin_name, GError **error_r)
{
const struct config_param *param = NULL;
while ((param = config_get_next_param(CONF_INPUT, param)) != NULL) {
const char *name =
config_get_block_string(param, "plugin", NULL);
- if (name == NULL)
- g_error("input configuration without 'plugin' name in line %d",
- param->line);
+ if (name == NULL) {
+ g_set_error(error_r, input_quark(), 0,
+ "input configuration without 'plugin' name in line %d",
+ param->line);
+ return NULL;
+ }
if (strcmp(name, plugin_name) == 0)
return param;
@@ -50,20 +60,35 @@ input_plugin_config(const char *plugin_name)
return NULL;
}
-void input_stream_global_init(void)
+bool
+input_stream_global_init(GError **error_r)
{
+ GError *error = NULL;
+
for (unsigned i = 0; input_plugins[i] != NULL; ++i) {
const struct input_plugin *plugin = input_plugins[i];
const struct config_param *param =
- input_plugin_config(plugin->name);
+ input_plugin_config(plugin->name, &error);
+ if (param == NULL && error != NULL) {
+ g_propagate_error(error_r, error);
+ return false;
+ }
if (!config_get_block_bool(param, "enabled", true))
/* the plugin is disabled in mpd.conf */
continue;
- if (plugin->init == NULL || plugin->init(param))
+ if (plugin->init == NULL || plugin->init(param, &error))
input_plugins_enabled[i] = true;
+ else {
+ g_propagate_prefixed_error(error_r, error,
+ "Failed to initialize input plugin '%s': ",
+ plugin->name);
+ return false;
+ }
}
+
+ return true;
}
void input_stream_global_finish(void)