WIP: add AlignConsecutiveDeclarations: true to .clang-format, and apply...
The AlignConsecutiveDeclarations was not defined in the .clang-format file, because it was 'only' introduced with clang-format-3.9, which, when .clang-format was first written, was not yet available....
Merge request reports
Activity
Note: this is WIP in order to not be picked up in any nightly builds, as it will surely cause conflicts with future merge request. However, I think we should consider to make this change at the time of the next release, and this MR should be seen as a starting point for that (and hopefully it won't be the start of a flame-war)
aligning forces changes to be bigger than they could be. Then again, it's already done for assignments in the current format...
Edited by Rosen Matevadded 205 commits
-
2c913b7a...58146ecb - 203 commits from branch
gaudi:master - 108179d9 - add AlignConsecutiveDeclarations: true to .clang-format
- 6d5b79c5 - Apply formatting
-
2c913b7a...58146ecb - 203 commits from branch
assigned to @clemenci
changed milestone to %v30r3
added code cleanup label
- Resolved by Gerhard Raven
@graven, you should remove the commit that applies the new formatting: I'll do it by hand just before merging.
added 119 commits
-
6d5b79c5...cb38e5ac - 118 commits from branch
gaudi:master - 9252455c - add AlignConsecutiveDeclarations: true to .clang-format
-
6d5b79c5...cb38e5ac - 118 commits from branch
added 5 commits
-
9252455c...4e61badc - 4 commits from branch
gaudi:master - f206dc88 - add AlignConsecutiveDeclarations: true to .clang-format
-
9252455c...4e61badc - 4 commits from branch
mentioned in merge request !651 (merged)
Included in !651 (merged)
