[effects/slidingpopups] Use range-based for loops
ClosedPublic

Authored by zzag on Jul 13 2018, 7:01 AM.

Diff Detail

Repository
R108 KWin
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
zzag created this revision.Jul 13 2018, 7:01 AM
Restricted Application added a project: KWin. · View Herald TranscriptJul 13 2018, 7:01 AM
Restricted Application added a subscriber: kwin. · View Herald Transcript
zzag requested review of this revision.Jul 13 2018, 7:01 AM
davidedmundson accepted this revision.Jul 13 2018, 10:47 AM
davidedmundson added a subscriber: davidedmundson.

Would have expected some fancy std::remove_if from you

This revision is now accepted and ready to land.Jul 13 2018, 10:47 AM
zzag added a comment.Jul 13 2018, 11:01 AM

Would have expected some fancy std::remove_if from you

Well, yeah, I could use std::copy_if. But I think that's fine to use loops here.

This revision was automatically updated to reflect the committed changes.