[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