Merge branch 'ak/config-bad-bool-error'
The error message given when a configuration variable that is expected to have a boolean value has been improved. * ak/config-bad-bool-error: config: improve error message for boolean config
This commit is contained in:
20
config.c
20
config.c
@ -1180,6 +1180,20 @@ static void die_bad_number(const char *name, const char *value)
|
||||
}
|
||||
}
|
||||
|
||||
NORETURN
|
||||
static void die_bad_bool(const char *name, const char *value)
|
||||
{
|
||||
if (!strcmp(name, "GIT_TEST_GETTEXT_POISON"))
|
||||
/*
|
||||
* We explicitly *don't* use _() here since it would
|
||||
* cause an infinite loop with _() needing to call
|
||||
* use_gettext_poison().
|
||||
*/
|
||||
die("bad boolean config value '%s' for '%s'", value, name);
|
||||
else
|
||||
die(_("bad boolean config value '%s' for '%s'"), value, name);
|
||||
}
|
||||
|
||||
int git_config_int(const char *name, const char *value)
|
||||
{
|
||||
int ret;
|
||||
@ -1252,8 +1266,10 @@ int git_config_bool_or_int(const char *name, const char *value, int *is_bool)
|
||||
|
||||
int git_config_bool(const char *name, const char *value)
|
||||
{
|
||||
int discard;
|
||||
return !!git_config_bool_or_int(name, value, &discard);
|
||||
int v = git_parse_maybe_bool(value);
|
||||
if (v < 0)
|
||||
die_bad_bool(name, value);
|
||||
return v;
|
||||
}
|
||||
|
||||
int git_config_string(const char **dest, const char *var, const char *value)
|
||||
|
Reference in New Issue
Block a user