Pass NULL instead of empty string to option parser

A bug, introduced in #576. If alignment isn't set in the config, the
settings would print a warning about an invalid value. But actually it's
just the default string, which can't get parsed.

The fallback value should get handled by a false return value and not by
the option_get_string's default value.
This commit is contained in:
Benedikt Heine 2019-01-09 13:07:08 +01:00
parent 6a8f1c52c3
commit 5627d06070

View File

@ -277,7 +277,7 @@ void load_settings(char *cmdline_config_path)
{ {
char *c = option_get_string( char *c = option_get_string(
"global", "global",
"alignment", "-align/-alignment", "", "alignment", "-align/-alignment", NULL,
"Text alignment left/center/right" "Text alignment left/center/right"
); );