diff options
author | Junio C Hamano <gitster@pobox.com> | 2016-02-26 13:37:27 -0800 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2016-02-26 13:37:27 -0800 |
commit | 0f0dd370c84b4068bd80cbab30cb2ea67abedbe0 (patch) | |
tree | 9200b08684af983b6bb34017e1d26471b31b3178 /Documentation/CodingGuidelines | |
parent | Merge branch 'dt/initial-ref-xn-commit-doc' (diff) | |
parent | add DEVELOPER makefile knob to check for acknowledged warnings (diff) | |
download | tgif-0f0dd370c84b4068bd80cbab30cb2ea67abedbe0.tar.xz |
Merge branch 'ls/makefile-cflags-developer-tweak'
There is a new DEVELOPER knob that enables many compiler warning
options in the Makefile.
* ls/makefile-cflags-developer-tweak:
add DEVELOPER makefile knob to check for acknowledged warnings
Diffstat (limited to 'Documentation/CodingGuidelines')
-rw-r--r-- | Documentation/CodingGuidelines | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/Documentation/CodingGuidelines b/Documentation/CodingGuidelines index c6e536f180..0ddd36879a 100644 --- a/Documentation/CodingGuidelines +++ b/Documentation/CodingGuidelines @@ -171,6 +171,11 @@ For C programs: - We try to keep to at most 80 characters per line. + - As a Git developer we assume you have a reasonably modern compiler + and we recommend you to enable the DEVELOPER makefile knob to + ensure your patch is clear of all compiler warnings we care about, + by e.g. "echo DEVELOPER=1 >>config.mak". + - We try to support a wide range of C compilers to compile Git with, including old ones. That means that you should not use C99 initializers, even if a lot of compilers grok it. |