D22371: Run clang-tidy with modernize-use-override check

[Roman Gilg]

romangg added a comment.


  I'm in favor of doing this. The problem with having this old style in code is not only warnings, but also one normally looks for examples in the code to write new code. Having old style in there leads to people using the old style for new code and then again the perpetuation of old style usage and/or prolonged reviews.
  
  Since this change is so large maybe we want to wait till sprint though to do it in a coordinated fashion with potential other code style updates.

REPOSITORY
  R108 KWin

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

To: zzag, #kwin
Cc: romangg, kwin, LeGast00n, sbergeron, jraleigh, fbampaloukas, GB_2, mkulinski, ragreen, jackyalcine, Pitel, iodelay, crozbo, bwowk, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwin/attachments/20190710/df2d7a29/attachment.html>