Review Request 122988: make PlasmaCore.ColorScope work better as advertised

Marco Martin notmart at gmail.com
Wed Mar 18 17:43:18 UTC 2015


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/122988/
-----------------------------------------------------------

(Updated March 18, 2015, 5:43 p.m.)


Review request for Plasma.


Repository: plasma-framework


Description
-------

Since attached properties cannot ever change during the lifetime of the object, a colorscope needs to be created for each obect that asks for the attached property.
now each instance if (internal) has m_inherit (only those created as an attached property), will search for the property of the proper ancestor.
Not super efficient, but the proeprties should be actually asked for rarely enough.

doesn't look much nice, but seems to work pretty reliably and the only way to.


Diffs (updated)
-----

  src/declarativeimports/core/colorscope.h fa75b9d 
  src/declarativeimports/core/colorscope.cpp cca0d2f 
  src/declarativeimports/core/framesvgitem.h 20486c7 
  src/declarativeimports/core/framesvgitem.cpp e2a0789 

Diff: https://git.reviewboard.kde.org/r/122988/diff/


Testing
-------


Thanks,

Marco Martin

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150318/23df49a3/attachment.html>


More information about the Plasma-devel mailing list