Differential D13837

Lookup loaded effects only once when building support information
ClosedPublic
Actions

Authored by zzag on Jul 2 2018, 8:15 AM.

Details

Reviewers
davidedmundson
Group Reviewers
KWin
Commits
R108:e7d6f12bbaf4: Lookup loaded effects only once when building support information
Test Plan

Ran qdbus org.kde.KWin /KWin supportInformation

Diff Detail

Repository
R108 KWin
Branch
libkwineffects-impl-supportinformation
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 546
Build 558: arc lint + arc unit
zzag created this revision.Jul 2 2018, 8:15 AM
Restricted Application added a project: KWin. · View Herald TranscriptJul 2 2018, 8:15 AM
Restricted Application added a subscriber: kwin. · View Herald Transcript
zzag requested review of this revision.Jul 2 2018, 8:15 AM
davidedmundson accepted this revision.Jul 2 2018, 8:40 AM
davidedmundson added a subscriber: davidedmundson.

Not exactly a hotspot.

This revision is now accepted and ready to land.Jul 2 2018, 8:40 AM
zzag added a comment.Jul 2 2018, 8:52 AM
In D13837#285903, @davidedmundson wrote:

Not exactly a hotspot.

Yeah, but that doesn't mean we should stop writing more optimal code. :-)

Closed by commit R108:e7d6f12bbaf4: Lookup loaded effects only once when building support information (authored by zzag). · Explain WhyJul 2 2018, 9:47 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Meffects.cpp (27 lines)

Diff 37029

View Options

effects.cpp

Loading...