Code

SubmittingPatches: mention older C compiler compatibility
authorJohannes Schindelin <Johannes.Schindelin@gmx.de>
Mon, 21 May 2007 12:48:49 +0000 (13:48 +0100)
committerJunio C Hamano <junkio@cox.net>
Tue, 22 May 2007 01:35:37 +0000 (18:35 -0700)
We do not appreciate C99 initializers, declarations after statements,
or "0" instead of "NULL".

Signed-off-by: Johannes Schindelin <Johannes.Schindelin@gmx.de>
Signed-off-by: Junio C Hamano <junkio@cox.net>
Documentation/SubmittingPatches

index 8cf5093dd9b51881977437a327fe69cad12e7efd..1ba9041b5b33d8830d5adf60e67e0f8ee9e48b3d 100644 (file)
@@ -62,6 +62,19 @@ in templates/hooks--pre-commit.  To help ensure this does not happen,
 run git diff --check on your changes before you commit.
 
 
+(1a) Try to be nice to older C compilers
+
+We try to support wide range of C compilers to compile
+git with. That means that you should not use C99 initializers, even
+if a lot of compilers grok it.
+
+Also, variables have to be declared at the beginning of the block
+(you can check this with gcc, using the -Wdeclaration-after-statement
+option).
+
+Another thing: NULL pointers shall be written as NULL, not as 0.
+
+
 (2) Generate your patch using git tools out of your commits.
 
 git based diff tools (git, Cogito, and StGIT included) generate