Differential D13118

[effects/blur] Lookup window once when disconnecting from blurChanged
ClosedPublic
Actions

Authored by zzag on May 25 2018, 4:58 PM.

Details

Reviewers
davidedmundson
Group Reviewers
KWin
Commits
R108:95ee05244299: [effects/blur] Lookup window once when disconnecting from blurChanged
Summary

When disconnecting from blurChanged signal, (w, connection) pair is
being looked up three times. We can do better by using QMap in a more
STL-like way.

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.May 25 2018, 4:58 PM
Restricted Application added a project: KWin. · View Herald TranscriptMay 25 2018, 4:58 PM
Restricted Application added a subscriber: kwin. · View Herald Transcript
zzag requested review of this revision.May 25 2018, 4:58 PM
davidedmundson accepted this revision.May 27 2018, 10:46 PM
This revision is now accepted and ready to land.May 27 2018, 10:46 PM
Closed by commit R108:95ee05244299: [effects/blur] Lookup window once when disconnecting from blurChanged (authored by zzag). · Explain WhyMay 31 2018, 9:08 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Meffects/blur/blur.cpp (8 lines)

Diff 35236

View Options

effects/blur/blur.cpp

Loading...