You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
It seems like this project is not using a .clang-format style file for formatting. Using make indent (which calls astyle with some options) wants to re-format basically the whole source tree for me.
@joto what's your take on this? Also what do you think about documenting this in the dev docs?
The text was updated successfully, but these errors were encountered:
Years ago I used astyle, but haven't used it in a long time. So it is a sort-of left-over from before as you can see by the fact that it is defined in the Makefile and not in the CMake config. Astyle seems to be unmaintained and doesn't work well with modern C++, so that is a dead end there. I wanted to switch to clang-format, but getting it to do what I want isn't that easy if at all possible. Only rather new versions are usable at all. So for the time being I am stuck with manual formatting. Now that more and more other people are contributing code, this should probably be "fixed".
I experimented with clang-format-3.8 but aren't very happy. It is rather buggy and doesn't support common formatting choices like indented nested preprocessor directives.
It seems like this project is not using a
.clang-format
style file for formatting. Usingmake indent
(which callsastyle
with some options) wants to re-format basically the whole source tree for me.@joto what's your take on this? Also what do you think about documenting this in the dev docs?
The text was updated successfully, but these errors were encountered: