git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH] CodingGuidelines: add a note to help contributors catch some errors earlier.

From
Yann Dirson <ydirson@altern.org>
Date
Oct 23, 2010, 14:17 UTC
Message-ID
<1287843434-10890-1-git-send-email-ydirson@altern.org>

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@altern.org>
---
 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
Next: Thiago Farina
Message 1 of 3 in “CodingGuidelines: add a note to help contributors catch some errors earlier.”
  1. CodingGuidelines: add a note to help contributors catch some errors earlier.Yann Dirson, Oct 23, 2010
  2. Thiago FarinaOct 23, 2010
  3. Yann DirsonOct 23, 2010

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.