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