[bug report] checkpatch: if statement does not need to be enclosed in parentheses
From: Alfonso Lima
Date: Sat Apr 08 2017 - 12:07:41 EST
Hi,
In current linux-next, if you run the script on this piece of
code:
#define FOO(a) \
if (a) { \
something(); \
something_else(); \
}
You should get:
ERROR: Macros with complex values should be enclosed in parentheses
#1: FILE: foo.c:1:
+#define FOO(a) \
+ if (a) { \
+ something(); \
+ something_else(); \
+ }
We could silence checkpatch.pl using "do {} while ()" around the
if statement. However, the "if () {}" statement should be
enough. If someone could confirm this, I'll go and fix it.
Thanks,
Alfonso