aboutsummaryrefslogtreecommitdiffstats
path: root/src/output_init.c
diff options
context:
space:
mode:
authorMax Kellermann <max@duempel.org>2009-03-01 13:31:56 +0100
committerMax Kellermann <max@duempel.org>2009-03-01 13:31:56 +0100
commitf298fcf3a651b9c072f1fa7242b4ff30141c6fe5 (patch)
treee5a0b55fa7c4ec21ee02c696cee902fa95ce441e /src/output_init.c
parentcb942eeb45e15d6b471625629dcb324e625cfac7 (diff)
downloadmpd-f298fcf3a651b9c072f1fa7242b4ff30141c6fe5.tar.gz
mpd-f298fcf3a651b9c072f1fa7242b4ff30141c6fe5.tar.xz
mpd-f298fcf3a651b9c072f1fa7242b4ff30141c6fe5.zip
output_init: return GError on error
Do error handling with GError instead of aborting with g_error().
Diffstat (limited to '')
-rw-r--r--src/output_init.c34
1 files changed, 15 insertions, 19 deletions
diff --git a/src/output_init.c b/src/output_init.c
index 4c052d943..c938d7eeb 100644
--- a/src/output_init.c
+++ b/src/output_init.c
@@ -42,7 +42,7 @@
}
static const struct audio_output_plugin *
-audio_output_detect(void)
+audio_output_detect(GError **error)
{
const struct audio_output_plugin *plugin;
unsigned i;
@@ -59,17 +59,19 @@ audio_output_detect(void)
return plugin;
}
+ g_set_error(error, audio_output_quark(), 0,
+ "Unable to detect an audio device");
return NULL;
}
bool
-audio_output_init(struct audio_output *ao, const struct config_param *param)
+audio_output_init(struct audio_output *ao, const struct config_param *param,
+ GError **error)
{
const char *name = NULL;
char *format = NULL;
struct block_param *bp = NULL;
const struct audio_output_plugin *plugin = NULL;
- GError *error = NULL;
if (param) {
const char *type = NULL;
@@ -80,18 +82,18 @@ audio_output_init(struct audio_output *ao, const struct config_param *param)
plugin = audio_output_plugin_get(type);
if (plugin == NULL) {
- g_error("couldn't find audio output plugin for type "
- "\"%s\" at line %i\n", type, param->line);
+ g_set_error(error, audio_output_quark(), 0,
+ "No such audio output plugin: %s",
+ type);
+ return false;
}
} else {
g_warning("No \"%s\" defined in config file\n",
CONF_AUDIO_OUTPUT);
- plugin = audio_output_detect();
- if (plugin == NULL) {
- g_warning("Unable to detect an audio device");
+ plugin = audio_output_detect(error);
+ if (plugin == NULL)
return false;
- }
g_message("Successfully detected a %s audio device",
plugin->name);
@@ -111,10 +113,9 @@ audio_output_init(struct audio_output *ao, const struct config_param *param)
bool ret;
ret = audio_format_parse(&ao->config_audio_format, format,
- &error);
+ error);
if (!ret)
- g_error("error parsing format at line %i: %s",
- bp->line, error->message);
+ return false;
} else
audio_format_clear(&ao->config_audio_format);
@@ -124,14 +125,9 @@ audio_output_init(struct audio_output *ao, const struct config_param *param)
ao->data = ao_plugin_init(plugin,
format ? &ao->config_audio_format : NULL,
- param, &error);
- if (ao->data == NULL) {
- g_warning("Failed to initialize \"%s\" [%s]: %s",
- ao->name, ao->plugin->name,
- error->message);
- g_error_free(error);
+ param, error);
+ if (ao->data == NULL)
return false;
- }
return true;
}