diff options
author | Ævar Arnfjörð Bjarmason <avarab@gmail.com> | 2018-08-17 13:02:50 +0000 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2018-08-17 10:08:56 -0700 |
commit | ce528de023f82d767096b0342b6da8039feff7bc (patch) | |
tree | 53e85fcee37cfd8b2611c47915c3acbb53143d00 /blame.c | |
parent | 53f9a3e157dbbc901a02ac2c73346d375e24978c (diff) | |
download | git-ce528de023f82d767096b0342b6da8039feff7bc.tar.gz |
refactor various if (x) FREE_AND_NULL(x) to just FREE_AND_NULL(x)
Change the few conditional uses of FREE_AND_NULL(x) to be
unconditional. As noted in the standard[1] free(NULL) is perfectly
valid, so we might as well leave this check up to the C library.
1. http://pubs.opengroup.org/onlinepubs/9699919799/functions/free.html
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'blame.c')
-rw-r--r-- | blame.c | 4 |
1 files changed, 1 insertions, 3 deletions
@@ -313,9 +313,7 @@ static void fill_origin_blob(struct diff_options *opt, static void drop_origin_blob(struct blame_origin *o) { - if (o->file.ptr) { - FREE_AND_NULL(o->file.ptr); - } + FREE_AND_NULL(o->file.ptr); } /* |