Conversation
9a0797e to
4630432
Compare
|
@nucleogenic do you have a preference of tab indentation vs. space indentation? The majority of C++ files use tab indentation right now, but there's a sizable chunk of space indented code too. Whatever we do there's going to be a huge diff. |
This reverts a previous change where configurations for the Eclipse IDE specifically was introduced. I think it's more important to have an IDE agnostic formatting framework, which clang-format provides Example command for formatting all C++ source code, as run from within the cpp dir: find . -iname '*.h' -o -iname '*.cpp' | xargs clang-format -i
4630432 to
b509d1c
Compare
|
|
I will postpone the wholesale reformatting of the codebase to another time. After reformatting the code does compile and sonarqube doesn't complain about new issues, so it should be fine. I had to add two rules compared to the original by @akuker -- namely Future improvements include adding a formatting check to CI, introduce git hooks and helper scripts. |



This reverts a previous change where configurations for the Eclipse IDE specifically was introduced.
I think it's more important to have an IDE agnostic formatting framework, which clang-format provides
Example command for formatting all C++ source code, as run from within the cpp dir:
find . -iname '*.h' -o -iname '*.cpp' | xargs clang-format -i