diff options
| author | Junio C Hamano <gitster@pobox.com> | 2024-08-26 11:32:24 -0700 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2024-08-26 11:32:24 -0700 |
| commit | 27d4f4032e8fc0e5c2f001af01b4102b1b4d2e9f (patch) | |
| tree | d05fdd2e0b51c2e0bb005f2f5ffeb81bba75b3f8 /Documentation/CodingGuidelines | |
| parent | 3222718ad7d9dfc50e31037bf2f3977d2071fb75 (diff) | |
| parent | 44db6f75cce574a0e410df5be61d40f28ec16f0a (diff) | |
Merge branch 'jc/coding-style-c-operator-with-spaces'
Write down whitespacing rules around C opeators.
* jc/coding-style-c-operator-with-spaces:
CodingGuidelines: spaces around C operators
Diffstat (limited to 'Documentation/CodingGuidelines')
| -rw-r--r-- | Documentation/CodingGuidelines | 11 |
1 files changed, 10 insertions, 1 deletions
diff --git a/Documentation/CodingGuidelines b/Documentation/CodingGuidelines index e4bd0abdcd..ccaea39752 100644 --- a/Documentation/CodingGuidelines +++ b/Documentation/CodingGuidelines @@ -303,7 +303,9 @@ For C programs: v12.01, 2022-03-28). - Variables have to be declared at the beginning of the block, before - the first statement (i.e. -Wdeclaration-after-statement). + the first statement (i.e. -Wdeclaration-after-statement). It is + encouraged to have a blank line between the end of the declarations + and the first statement in the block. - NULL pointers shall be written as NULL, not as 0. @@ -323,6 +325,13 @@ For C programs: while( condition ) func (bar+1); + - A binary operator (other than ",") and ternary conditional "?:" + have a space on each side of the operator to separate it from its + operands. E.g. "A + 1", not "A+1". + + - A unary operator (other than "." and "->") have no space between it + and its operand. E.g. "(char *)ptr", not "(char *) ptr". + - Do not explicitly compare an integral value with constant 0 or '\0', or a pointer value with constant NULL. For instance, to validate that counted array <ptr, cnt> is initialized but has no elements, write: |
