lobster icon indicating copy to clipboard operation
lobster copied to clipboard

clang-format

Open jacereda opened this issue 6 years ago • 2 comments

Hi,

Could you clang-format the code under dev/src? I want to PR some changes and I don't want to pollute it with formatting stuff.

Also, I guess the .clang-format file should be moved to dev/src, since you probably don't want to clang-format external dependencies.

jacereda avatar Apr 29 '19 11:04 jacereda

There's a fair bit of code that would need // clang-format off comments, in particular for pre-processor directives (there is partial support for formatting these with indentation, but some is missing, see https://bugs.llvm.org/show_bug.cgi?id=36020) before we can clang-format all of it.

For now, I'd prefer if you simply used git clang-format to just format your new code.

aardappel avatar Apr 29 '19 14:04 aardappel

Ah, and yes that file should be moved :)

aardappel avatar Apr 29 '19 14:04 aardappel