This reverts commit d38565f64e.
It added blank lines between one line member function definitions
inside class bodies/header files, something which is undesirable.
With the current code it makes no difference, but it will force new
definitions to be in its own block, e.g. we cannot define two
functions without a blank line between then.
Allows to format the code in better ways adjusting the comments to fit
the column limit. In few cases the comment will require to be moved to
other place (e.g. at the top of a statement rather than to the side of
it).
With this we prefer to format a function declaration as:
int function(arg1,
arg2,
arg3);
Rather than:
int function(
arg1, arg2, arg3);
Cons: This makes the definition of NewParams structures a bit worse.
With this we prefer to format function calls as:
int value = function(arg1,
arg2,
arg3);
Or:
int value =
function(arg1, arg2, arg3);
Rather than:
int value = function(
arg1, arg2, arg3);
Making PenaltyBreakAssignment bigger creates better looking
const/variable declarations, where the following term to the
assignment operator is in the same line.
Maybe it's not the final version, but it's a good start. We cannot
match the coding style that we were using, but it's pretty close.
We'll start applying clang-format incrementally in a near future.
David Capello
Christian Kaiser
David Capello