Run clang-format
This MR is better read commit per commit.
The first four commits fix some problematic code, to work properly after clang-format, while the fifth one is the actual run.
The fifth commit is easy to run again, so maybe we’d want to merge the first four first, merge a bunch of other MRs to avoid having to run them again after that, and then re-run clang-format and make the CI enforce it afterwards.
I’d also like to make the CI enforce that all commits pass the clang-format check, but I’m not too sure how to go with that without it being too annoying to contributors, this is open to discussion, with no change for now.
Edited by Jonathan Neuhauser