diff options
author | René Scharfe <l.s.r@web.de> | 2016-09-29 17:29:29 +0200 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2016-09-29 15:42:18 -0700 |
commit | 76dd98c13947bd811c1d11d0c63ccdc9a4fb3142 (patch) | |
tree | 71fe0bb6d2dafa7df4986c3cdd742d8f081324ad /contrib | |
parent | 9ed0d8d6e6de7737fe9a658446318b86e57c6fad (diff) | |
download | git-76dd98c13947bd811c1d11d0c63ccdc9a4fb3142.tar.gz git-76dd98c13947bd811c1d11d0c63ccdc9a4fb3142.tar.xz |
remove unnecessary check before QSORT
Add a semantic patch for removing checks similar to the one that QSORT
already does internally and apply it to 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/qsort.cocci | 18 |
1 files changed, 18 insertions, 0 deletions
diff --git a/contrib/coccinelle/qsort.cocci b/contrib/coccinelle/qsort.cocci index a094e7c5e..22b93a996 100644 --- a/contrib/coccinelle/qsort.cocci +++ b/contrib/coccinelle/qsort.cocci @@ -17,3 +17,21 @@ expression nmemb, compar; @@ - qsort(base, nmemb, sizeof(T), compar); + QSORT(base, nmemb, compar); + +@@ +expression base, nmemb, compar; +@@ +- if (nmemb) + QSORT(base, nmemb, compar); + +@@ +expression base, nmemb, compar; +@@ +- if (nmemb > 0) + QSORT(base, nmemb, compar); + +@@ +expression base, nmemb, compar; +@@ +- if (nmemb > 1) + QSORT(base, nmemb, compar); |