It is a bit of a shame to wait for other's review before noticing decl-after-stmt situations, whereas a quick check could have found it. There are probably a number of other autoamted checks which could be described here, but that's a start. Signed-off-by: Yann Dirson <ydirson@xxxxxxxxxx> --- Documentation/CodingGuidelines | 5 +++++ 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/Documentation/CodingGuidelines b/Documentation/CodingGuidelines index 09ffc46..7ecd4f3 100644 --- a/Documentation/CodingGuidelines +++ b/Documentation/CodingGuidelines @@ -139,3 +139,8 @@ For C programs: - When we pass <string, length> pair to functions, we should try to pass them in that order. + + - C code is expected to compile when the build is launched with the + following command: + + make CFLAGS="-Wall -Wdeclaration-after-statement -Werror" -- 1.7.2.3 -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html