Attention is currently required from: Hoernchen, laforge.
osmith has posted comments on this change by Hoernchen. ( https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/42330?usp=email )
Change subject: fw source: force reformat once and for all ......................................................................
Patch Set 2:
(2 comments)
Patchset:
PS1:
jenkins was using clang-format 14 from debian 12. […]
Done
PS1:
clang-format formats and that is the end of it, no discussions, NOTHING EVER AGAIN. […]
The documented process for posting kernel patches tells the user that they should run checkpatch, while clang-format is not mentioned: https://www.kernel.org/doc/html/latest/process/5.Posting.html
There is a clang-format file, but its formatting is clearly not applied through the whole codebase. When I run clang-format on current linux.git master, I get a diff of 29 million lines.
``` $ git diff | wc -l 29068766 ```
How about we discuss this in a jitsi call, e.g. after tomorrow's team meeting?