From bc0fec0afe8e541dd2d7df2ddbe04b13f6a93678 Mon Sep 17 00:00:00 2001 From: Max Kellermann Date: Fri, 9 Sep 2011 21:16:46 +0200 Subject: conf: move duplicate check to _read_name_value() config_add_block_param() cannot fail, which makes it easier to use. --- src/conf.c | 27 +++++++++++++-------------- 1 file changed, 13 insertions(+), 14 deletions(-) (limited to 'src/conf.c') diff --git a/src/conf.c b/src/conf.c index ee5aaceb3..47a3bae9c 100644 --- a/src/conf.c +++ b/src/conf.c @@ -223,20 +223,13 @@ void config_global_check(void) } } -bool +void config_add_block_param(struct config_param * param, const char *name, - const char *value, int line, GError **error_r) + const char *value, int line) { struct block_param *bp; - bp = config_get_block_param(param, name); - if (bp != NULL) { - g_set_error(error_r, config_quark(), 0, - "\"%s\" first defined on line %i, and " - "redefined on line %i\n", name, - bp->line, line); - return false; - } + assert(config_get_block_param(param, name) == NULL); param->num_block_params++; @@ -250,8 +243,6 @@ config_add_block_param(struct config_param * param, const char *name, bp->value = g_strdup(value); bp->line = line; bp->used = false; - - return true; } static bool @@ -283,8 +274,16 @@ config_read_name_value(struct config_param *param, char *input, unsigned line, return false; } - return config_add_block_param(param, name, value, line, - error_r); + const struct block_param *bp = config_get_block_param(param, name); + if (bp != NULL) { + g_set_error(error_r, config_quark(), 0, + "\"%s\" is duplicate, first defined on line %i", + name, bp->line); + return false; + } + + config_add_block_param(param, name, value, line); + return true; } static struct config_param * -- cgit v1.2.3