From 493384e39c1ddfcc4badf962706e02302577d89f Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Carlos=20Mart=C3=ADn=20Nieto?= Date: Thu, 7 Apr 2011 11:24:16 +0200 Subject: [PATCH] config: make cvar_free behave more like other free functions MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Make cvar_free return void instad of the next element, as it was mostly a hack to make cvar_list_free shorter but it's now using the list macros. Also check if the input is NULL and return immediately in that case. Signed-off-by: Carlos Martín Nieto --- src/config.c | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/src/config.c b/src/config.c index d537fd8a5..5f0bcd880 100644 --- a/src/config.c +++ b/src/config.c @@ -37,15 +37,14 @@ static int config_parse(git_config *cfg_file); static int parse_variable(git_config *cfg, char **var_name, char **var_value); void git_config_free(git_config *cfg); -static git_cvar *cvar_free(git_cvar *var) +static void cvar_free(git_cvar *var) { - git_cvar *next = var->next; + if (var == NULL) + return; free(var->name); free(var->value); free(var); - - return next; } static void cvar_list_free(git_cvar_list *list)