aboutsummaryrefslogtreecommitdiff
path: root/contrib
diff options
context:
space:
mode:
authorRené Scharfe <l.s.r@web.de>2016-10-08 16:14:57 +0200
committerJunio C Hamano <gitster@pobox.com>2016-10-10 11:37:41 -0700
commit39ea59a2570547166834ceeff9ae0c0c05748f35 (patch)
treeb9874de8e3ca7b71f3ca23dd8f0a92fe210b2ec0 /contrib
parent353d84c537485500989b190c5af93ad224264e2c (diff)
downloadgit-39ea59a2570547166834ceeff9ae0c0c05748f35.tar.gz
git-39ea59a2570547166834ceeff9ae0c0c05748f35.tar.xz
remove unnecessary NULL check before free(3)
free(3) handles NULL pointers just fine. Add a semantic patch for removing unnecessary NULL checks before calling this function, and apply it on the code base. Signed-off-by: Rene Scharfe <l.s.r@web.de> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'contrib')
-rw-r--r--contrib/coccinelle/free.cocci5
1 files changed, 5 insertions, 0 deletions
diff --git a/contrib/coccinelle/free.cocci b/contrib/coccinelle/free.cocci
new file mode 100644
index 000000000..e28213161
--- /dev/null
+++ b/contrib/coccinelle/free.cocci
@@ -0,0 +1,5 @@
+@@
+expression E;
+@@
+- if (E)
+ free(E);