cvra / robot-software

CVRA monorepo - All software running on our bots lives here
MIT License
44 stars 21 forks source link

Switch to clang-format #176

Closed antoinealb closed 5 years ago

antoinealb commented 5 years ago

I added support for clang-format and ran it on the whole source tree. Uncrustify that we have been using so far is slowly dying, its output is unstable from version to version and it has less capabilities than clang-format. I tried not to change too much the code, but due to our inconsistent formatting, some changes were needed anyway.

Reasons for doing this:

Reasons for not doing this:

Note that this was tested on an install of Clang 7, make sure you have a recent enough version!

SyrianSpock commented 5 years ago

Please wait for #175 to be merged before merging this.

antoinealb commented 5 years ago

Rebase now that #175 landed. Pinging @SyrianSpock for review 👍