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

Reply via email to