summaryrefslogtreecommitdiffstats
path: root/contrib/coccinelle/preincr.cocci
diff options
context:
space:
mode:
authorJunio C Hamano <gitster@pobox.com>2018-10-24 03:10:10 +0200
committerJunio C Hamano <gitster@pobox.com>2018-10-24 03:10:10 +0200
commit05b4ed61f4e669da839892e03dad70e15d9a4cd3 (patch)
treeda7c3e04ce9309a67086e5a662acd8019e733f4f /contrib/coccinelle/preincr.cocci
parentGit 2.19.1 (diff)
downloadgit-05b4ed61f4e669da839892e03dad70e15d9a4cd3.tar.xz
git-05b4ed61f4e669da839892e03dad70e15d9a4cd3.zip
cocci: simplify "if (++u > 1)" to "if (u++)"
It is more common to use post-increment than pre-increment when the side effect is the primary thing we want in our code and in C in general (unlike C++). Initializing a variable to 0, incrementing it every time we do something, and checking if we have already done that thing to guard the code to do that thing, is easier to understand when written if (u++) ; /* we've done that! */ else do_it(); /* just once. */ but if you try to use pre-increment, you end up with a less natural looking if (++u > 1) Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'contrib/coccinelle/preincr.cocci')
-rw-r--r--contrib/coccinelle/preincr.cocci5
1 files changed, 5 insertions, 0 deletions
diff --git a/contrib/coccinelle/preincr.cocci b/contrib/coccinelle/preincr.cocci
new file mode 100644
index 0000000000..7fe1e8d2d9
--- /dev/null
+++ b/contrib/coccinelle/preincr.cocci
@@ -0,0 +1,5 @@
+@ preincrement @
+identifier i;
+@@
+- ++i > 1
++ i++