zzag added a comment.

  Looks good to me. There are still many coding style issues. I suggest to run 
`clang-format` over individual files in this patch and then adjust the most 
problematic places (I don't think there are that many).
  
  You could borrow a `.clang-format` file from 
https://github.com/KDE/extra-cmake-modules/blob/master/kde-modules/clang-format.cmake.
 Just make sure that `ColumnLimit` is set to 0.

REPOSITORY
  R127 KWayland

REVISION DETAIL
  https://phabricator.kde.org/D26858

To: apol, #kwin, #frameworks, zzag
Cc: zzag, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

Reply via email to