path: root/help.c
diff options
authorÆvar Arnfjörð Bjarmason <>2017-06-15 23:15:49 (GMT)
committerJunio C Hamano <>2017-06-16 19:44:09 (GMT)
commit88ce3ef636b1385e861ec0e9e2155248b999b032 (patch)
tree0da315e544f734755c717c04852b2c44ac3345d3 /help.c
parente140f7afddcdce2bae062ea1578eac38c744e3a5 (diff)
*.[ch] refactoring: make use of the FREE_AND_NULL() macro
Replace occurrences of `free(ptr); ptr = NULL` which weren't caught by the coccinelle rule. These fall into two categories: - free/NULL assignments one after the other which coccinelle all put on one line, which is functionally equivalent code, but very ugly. - manually spotted occurrences where the NULL assignment isn't right after the free() call. Signed-off-by: Ævar Arnfjörð Bjarmason <> Signed-off-by: Junio C Hamano <>
Diffstat (limited to 'help.c')
1 files changed, 1 insertions, 2 deletions
diff --git a/help.c b/help.c
index db7f3d7..a2f48c0 100644
--- a/help.c
+++ b/help.c
@@ -267,9 +267,8 @@ static void add_cmd_list(struct cmdnames *cmds, struct cmdnames *old)
for (i = 0; i < old->cnt; i++)
cmds->names[cmds->cnt++] = old->names[i];
- free(old->names);
+ FREE_AND_NULL(old->names);
old->cnt = 0;
- old->names = NULL;
/* An empirically derived magic number */