diff options
| author | Junio C Hamano <junkio@cox.net> | 2006-08-27 21:19:39 -0700 |
|---|---|---|
| committer | Junio C Hamano <junkio@cox.net> | 2006-08-27 21:19:39 -0700 |
| commit | 4cac42b1324951579036a9d3ac403f5c2c3eeed8 (patch) | |
| tree | 5106e1aff2bd1d90fef4c26550b777dbe1142429 /path-list.c | |
| parent | b3c952f8386cebe12fc95227866683bb1cec99a9 (diff) | |
| download | git-4cac42b1324951579036a9d3ac403f5c2c3eeed8.tar.gz git-4cac42b1324951579036a9d3ac403f5c2c3eeed8.tar.xz | |
free(NULL) is perfectly valid.
Jonas noticed some places say "if (X) free(X)" which is totally
unnecessary.
Signed-off-by: Junio C Hamano <junkio@cox.net>
Diffstat (limited to 'path-list.c')
| -rw-r--r-- | path-list.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/path-list.c b/path-list.c index f15a10de3..b1ee72d1d 100644 --- a/path-list.c +++ b/path-list.c @@ -85,8 +85,7 @@ void path_list_clear(struct path_list *list, int free_items) for (i = 0; i < list->nr; i++) { if (list->strdup_paths) free(list->items[i].path); - if (list->items[i].util) - free(list->items[i].util); + free(list->items[i].util); } free(list->items); } |