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

Aleix Pol Gonzalez noreply at phabricator.kde.org
Wed Jul 10 14:52:13 BST 2019


apol added a comment.


  I did a similar patch along the lines years ago and it was rejected in grounds of git history pollution, which I think is a bad reason to restrict standard c++ usage.
  
  +1 from me

REPOSITORY
  R108 KWin

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

To: zzag, #kwin
Cc: apol, 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, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwin/attachments/20190710/8fb4bfba/attachment.html>


More information about the kwin mailing list