Revise coding style and documentation requirements
This is unsorted list:
-
opening curly braces on new line when the outside construct is multiline, e.g.
for (foo;
bar;
baz) {
vs current
for (foo;
bar;
baz)
{
-
Using parentheses to explicitly set priority in conditions, e.g.
((foo == TRUE) || (bar == FALSE))
vs current
(foo == TRUE || bar == FALSE)
-
Explicit NULL
orFALSE
comparison
(foo == FALSE && bar == NULL)
vs current
(!foo && !bar)
Edited by Ondřej Surý