D17898: Cast to AbstractClient instead of Client in Toplevel::setReadyForPainting

zzag added a comment.


  In D17898#384648 <https://phabricator.kde.org/D17898#384648>, @graesslin wrote:
  
  > I could imagine that something clang based would be able to cover such cases. So in case such a tool exists I would be all for reformat once again.
  
  
  Yes, there's a tool called clang-format[1]. Basically, we would need to create a style configuration file and then run
  
    find . -name '*.h' -o -name '*.cpp' -o -exec clang-format -i -style=file {} \;
  
  Also, what's really cool is that it's quite easy to integrate clang-format with Vim[2], Emacs, etc.
  
  [1] https://clang.llvm.org/docs/ClangFormat.html
  [2] http://clang.llvm.org/docs/ClangFormat.html#vim-integration
  [3] Qt Creator's .clang-format. https://github.com/qt-creator/qt-creator/blob/master/.clang-format
  [4] Qt's .clang-format. https://github.com/qt/qt5/blob/5.12/_clang-format

REPOSITORY
  R108 KWin

BRANCH
  ready-for-painting-cast

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

To: graesslin, #kwin, zzag
Cc: davidedmundson, zzag, kwin, GB_2, mkulinski, ragreen, jackyalcine, Pitel, iodelay, bwowk, ZrenBot, ngraham, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart