D23085: Make iterating over Layer enum simple

Frederik Gladhorn <[email protected]>
Newsgroups gmane.comp.kde.devel.kwin
Message-ID <[email protected]>
gladhorn marked an inline comment as done.
gladhorn added a comment.


  In D23085#510297 <https://phabricator.kde.org/D23085#510297>, @zzag wrote:
  
  > > to iterate over the enum, is mostly confusing
  >
  > I actually find it a bit cool because we're iterating through layers, but yeah this part can be simpler. :-)
  
  
  Yes, I was amused as well, but to understand what is going on I had to read code in two places instead of one.

INLINE COMMENTS

> zzag wrote in layers.cpp:536
> You don't need this static_cast.

Nice, of course :)

REPOSITORY
  R108 KWin

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

To: gladhorn, #kwin, romangg, zzag
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
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.