From 243bfd33997484ee8ea2450f1910306f7138193b Mon Sep 17 00:00:00 2001 From: Johannes Schindelin Date: Mon, 21 May 2007 13:48:49 +0100 Subject: [PATCH] SubmittingPatches: mention older C compiler compatibility We do not appreciate C99 initializers, declarations after statements, or "0" instead of "NULL". Signed-off-by: Johannes Schindelin Signed-off-by: Junio C Hamano --- Documentation/SubmittingPatches | 13 +++++++++++++ 1 file changed, 13 insertions(+) diff --git a/Documentation/SubmittingPatches b/Documentation/SubmittingPatches index 8cf5093dd..1ba9041b5 100644 --- a/Documentation/SubmittingPatches +++ b/Documentation/SubmittingPatches @@ -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 -- 2.26.2