Make the attached property a simple QObject
ClosedPublic

Authored by mart on Feb 6 2020, 7:42 PM.

Details

Summary

move all the logic in an attached property QObject
it's a bit less heavy to create so should make things slightly better
when this is used in delegates

Test Plan

Plasmoid background removal (with colors inversion) still works

Diff Detail

Repository
R242 Plasma Framework (Library)
Branch
phab/qobject
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 22148
Build 22166: arc lint + arc unit
mart created this revision.Feb 6 2020, 7:42 PM
Restricted Application added a project: Frameworks. · View Herald TranscriptFeb 6 2020, 7:42 PM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
mart requested review of this revision.Feb 6 2020, 7:42 PM
mart edited the test plan for this revision. (Show Details)Feb 6 2020, 7:45 PM
mart added reviewers: bshah, Plasma.
nicolasfella retitled this revision from Make the attached proeprty a simple QObject to Make the attached property a simple QObject.Feb 6 2020, 8:06 PM
apol added a subscriber: apol.Feb 6 2020, 10:54 PM

LGTM otherwise

src/declarativeimports/core/colorscope.cpp
57

let's make this a proper method? it's used twice here and a normal method should be quicker to use and easier on the reader.

This revision was not accepted when it landed; it landed in state Needs Review.Feb 10 2020, 3:50 PM
This revision was automatically updated to reflect the committed changes.

The component changes seem unrelated