We follow the Linux kernel coding style.
As a general rule :-
-
all indentation is tabs (set to 8 char) with the exception of continuation lines that are aligned with tabs and then spaces
-
all keywords followed by a '(' have a space in between
if (condition)
for (i = 0; i < 5; i++)
- function calls do NOT have a space between their name and argument
i = some_function(argument);
-
usually there is no space on the inside of parenthesis (see examples above)
-
function / method implementations have their opening curly braces in column 1
-
all other opening curly braces follow at the end of the line, with a space separating them:
if (condition) {
dosomething();
dosomethingelse();
}
- both sides of an if / else clause either use or do not use curly braces:
if (condition)
i = 4;
else
j = 6;
if (condition) {
i = 6;
} else {
i = 4;
j = 6;
}
- don't do assignments inside if () statements, always do it this way
res = something();
if (!res)
return NULL;
- if you don't need the result you can do
if (!something())
return NULL;
- use space to make visual separation easier
a = b + 3 + e / 4;
- continuation lines have the operator / comma at the end
if (very_long_conditiont_1 ||
condition_2)
b = a + (c + d +
f + z);
- switch statements with blocks are a little bit special (to avoid indenting too far)
switch (foo) {
case FIRST:
whatever();
break;
case SECOND: {
int i;
for (i = 0; i < 5; i++)
do_something(i);
}
}
-
comments should be C style not C++/C99
for single line comments
/* This is a single line comment */
for multi-line comments
/*
* This is a multi
* line comment
*/
-
variable declarations should be at the beginning of a code block, not interspersed in the middle
-
variable and function naming should be all lower case with _ used for spaces