From 209ea0657b7970b4b8069ce44b43eecb97681892 Mon Sep 17 00:00:00 2001 From: "Rafael G. Martins" Date: Tue, 20 Dec 2016 18:05:51 +0100 Subject: config-parser: improved error reporting, added bc_config_get_with_default --- src/common/config-parser.c | 23 +++++++++++++++++------ src/common/config-parser.h | 2 ++ 2 files changed, 19 insertions(+), 6 deletions(-) (limited to 'src/common') diff --git a/src/common/config-parser.c b/src/common/config-parser.c index 24fb299..ff05f2e 100644 --- a/src/common/config-parser.c +++ b/src/common/config-parser.c @@ -70,8 +70,8 @@ bc_config_parse(const char *src, size_t src_len, bc_error_t **err) continue; } if (err != NULL) - *err = bc_error_new_printf(BC_ERROR_CONFIG_PARSER, - "File must start with section"); + *err = bc_error_parser(BC_ERROR_CONFIG_PARSER, src, src_len, + current, "File must start with section."); break; case CONFIG_SECTION_START: @@ -92,8 +92,8 @@ bc_config_parse(const char *src, size_t src_len, bc_error_t **err) if (c != '\r' && c != '\n') break; if (err != NULL) - *err = bc_error_new_printf(BC_ERROR_CONFIG_PARSER, - "Section names can't have new lines"); + *err = bc_error_parser(BC_ERROR_CONFIG_PARSER, src, src_len, + current, "Section names can't have new lines."); break; case CONFIG_SECTION_KEY: @@ -109,8 +109,8 @@ bc_config_parse(const char *src, size_t src_len, bc_error_t **err) size_t end = is_last && c != '\n' && c != '\r' ? src_len : current; key = bc_strndup(src + start, end - start); - *err = bc_error_new_printf(BC_ERROR_CONFIG_PARSER, - "Key without value: %s", key); + *err = bc_error_parser(BC_ERROR_CONFIG_PARSER, src, src_len, + current, "Key without value: %s.", key); free(key); key = NULL; } @@ -225,6 +225,17 @@ bc_config_get(bc_config_t *config, const char *section, const char *key) } +const char* +bc_config_get_with_default(bc_config_t *config, const char *section, const char *key, + const char *default_) +{ + const char *rv = bc_config_get(config, section, key); + if (rv == NULL) + return default_; + return rv; +} + + void bc_config_free(bc_config_t *config) { diff --git a/src/common/config-parser.h b/src/common/config-parser.h index f9f287c..e249c37 100644 --- a/src/common/config-parser.h +++ b/src/common/config-parser.h @@ -22,6 +22,8 @@ char** bc_config_list_sections(bc_config_t *config); char** bc_config_list_keys(bc_config_t *config, const char *section); const char* bc_config_get(bc_config_t *config, const char *section, const char *key); +const char* bc_config_get_with_default(bc_config_t *config, const char *section, + const char *key, const char *default_); void bc_config_free(bc_config_t *config); #endif /* _CONFIG_PARSER_H */ -- cgit v1.2.3-18-g5258