Also use "git_config_string" to simplify code where "cmd" is set. Signed-off-by: Christian Couder <chriscool@xxxxxxxxxxxxx> --- diff.c | 7 ++----- 1 files changed, 2 insertions(+), 5 deletions(-) diff --git a/diff.c b/diff.c index 5640cbf..173bb61 100644 --- a/diff.c +++ b/diff.c @@ -57,7 +57,7 @@ static int parse_diff_color_slot(const char *var, int ofs) static struct ll_diff_driver { const char *name; struct ll_diff_driver *next; - char *cmd; + const char *cmd; } *user_diff, **user_diff_tail; /* @@ -86,10 +86,7 @@ static int parse_lldiff_command(const char *var, const char *ep, const char *val user_diff_tail = &(drv->next); } - if (!value) - return config_error_nonbool(var); - drv->cmd = xstrdup(value); - return 0; + return git_config_string(&(drv->cmd), var, value); } /* -- 1.5.4.1.129.gf12af-dirty - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html