diff options
author | Junio C Hamano <gitster@pobox.com> | 2008-02-11 10:41:18 -0800 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2008-02-11 13:11:36 -0800 |
commit | 40ea4ed9032a80c9dba706d6030bd11b08c35f4d (patch) | |
tree | e59d64db67bef5ef33ae220c445ce3dba0236b22 | |
parent | builtin-gc.c: guard config parser from value=NULL (diff) | |
download | tgif-40ea4ed9032a80c9dba706d6030bd11b08c35f4d.tar.xz |
Add config_error_nonbool() helper function
This is used to report misconfigured configuration file that does not
give any value to a non-boolean variable, e.g.
[section]
var
It is perfectly fine to say it if the section.var is a boolean (it means
true), but if a variable expects a string value it should be flagged as
a configuration error.
Signed-off-by: Junio C Hamano <gitster@pobox.com>
-rw-r--r-- | cache.h | 1 | ||||
-rw-r--r-- | config.c | 9 |
2 files changed, 10 insertions, 0 deletions
@@ -589,6 +589,7 @@ extern int git_config_set_multivar(const char *, const char *, const char *, int extern int git_config_rename_section(const char *, const char *); extern const char *git_etc_gitconfig(void); extern int check_repository_format_version(const char *var, const char *value); +extern int config_error_nonbool(const char *); #define MAX_GITNAME (1000) extern char git_default_email[MAX_GITNAME]; @@ -1079,3 +1079,12 @@ int git_config_rename_section(const char *old_name, const char *new_name) free(config_filename); return ret; } + +/* + * Call this to report error for your variable that should not + * get a boolean value (i.e. "[my] var" means "true"). + */ +int config_error_nonbool(const char *var) +{ + return error("Missing value for '%s'", var); +} |