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