<table><tr><td style="">romangg added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D22371">View Revision</a></tr></table><br /><div><div><p>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.</p>

<p>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.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R108 KWin</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D22371">https://phabricator.kde.org/D22371</a></div></div><br /><div><strong>To: </strong>zzag, KWin<br /><strong>Cc: </strong>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<br /></div>