D23011: Replace foreach loops and improve code style in composite
Vlad Zagorodniy
noreply at phabricator.kde.org
Thu Aug 8 14:34:07 BST 2019
zzag added a comment.
You forgot to address my comment about short names.
Short names go against the frameworks coding style.
"con" needs to become "connection"
"win" needs to become "window"
I know that old code is full of short names but let's at least follow the coding style in "new" code.
REPOSITORY
R108 KWin
REVISION DETAIL
https://phabricator.kde.org/D23011
To: romangg, #kwin
Cc: zzag, kwin, LeGast00n, sbergeron, jraleigh, fbampaloukas, GB_2, mkulinski, ragreen, jackyalcine, Pitel, iodelay, crozbo, bwowk, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, romangg, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwin/attachments/20190808/8ede7d65/attachment.html>
More information about the kwin
mailing list