D24826: Enforce 100 chars line width
Roman Gilg
noreply at phabricator.kde.org
Thu Sep 24 22:16:08 BST 2020
romangg added a comment.
FYI I'm using a strict 100 chars limit nowadays on all my KWinFT projects (with the exception of KWinFT itself for now) and I'm very happy with this decision.
As an example you can compare these two otherwise still pretty similar files:
https://invent.kde.org/plasma/libkscreen/-/blob/master/src/backendmanager.cpp
https://gitlab.com/kwinft/disman/-/blob/master/lib/backendmanager.cpp
I don't even think about any of this anymore, especially I don't care if clang-format moves a function name or puts a bracket here or there. I just apply clang-format to the file and that's it. The consistency trumps anything else.
Anyway closing this diff as requested.
REPOSITORY
R240 Extra CMake Modules
REVISION DETAIL
https://phabricator.kde.org/D24826
To: romangg, #frameworks, cullmann
Cc: carewolf, winterz, zzag, kde-frameworks-devel, kde-buildsystem, LeGast00n, cblack, bencreasy, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-buildsystem/attachments/20200924/46790c31/attachment.htm>
More information about the Kde-buildsystem
mailing list