Conversation
@marcan does the in-kernel .clang-format do the right thing*?
1
0
2
@marcan if it means fewer respins for checkpatch reporting minor stuff, or less time spent fixing it on behalf of people that's a win in my book. Curious how you get on!
1
0
1
Edited 1 year ago
@marcan that seems very close to sane! The corner that I am directly responsible for is small too, but I'll give it a try there (albeit with UT_Always). Thanks!
0
0
0
@llvm @marcan Right, but my perspective is a bit warped due to the "flaws" with the kernel's clangd integration.
0
0
0
@llvm @marcan Hmm, been playing with it a bit but have got stuck trying to figure out which option controls the following:
1
0
0
@marcan @llvm yah I prefer it without but not to the point that I'd not change. Thanks!
1
0
1
@ecurtin @llvm @marcan I played around with it some more & it keeps "doing the wrong thing" to the point that I didn't feel I could commit it.
I *think* I'll align the files that I'm responsible for to .clang-format & just run it myself as cleanup in merge commits.
0
0
0